Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate Mapper#mapPackage() #2419

Closed
evanchooly opened this issue Jun 9, 2023 · 0 comments
Closed

Deprecate Mapper#mapPackage() #2419

evanchooly opened this issue Jun 9, 2023 · 0 comments
Labels
docs enhancement migration Issue contains updates that should be noted in a migration document
Milestone

Comments

@evanchooly
Copy link
Member

evanchooly commented Jun 9, 2023

Move this to the config file and document it. The property should be named morphia.map-packages to support mapping multiple packages. If morphia.map-subpackages is set true, this should provide essential coverage of all mapping needs. If the element is missing this will result in a full classpath scan which is less than ideal and should be prominently documented.

Deprecate #map(Class...) and #map(List<Class>) as well. This effectively makes the package the only granularity for mapping entities.

@evanchooly evanchooly added enhancement docs migration Issue contains updates that should be noted in a migration document labels Jun 9, 2023
@evanchooly evanchooly added this to the 2.4.0 milestone Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs enhancement migration Issue contains updates that should be noted in a migration document
Projects
None yet
Development

No branches or pull requests

1 participant