-
Notifications
You must be signed in to change notification settings - Fork 87
Migrate to newest version of dozer #72
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
Comments
@AbhayChandel please comment the current status. If done then also assign milestone and close. |
I got the response via mail:
Agreed. Lets start with PR. |
Thanks to @Rohit90700 this is fixed via PR #86 |
I always thought we never made our fundamental features work with orika.
So if we are planning an incompatible change for beanmapping anyhow with |
just read https://www.baeldung.com/java-performance-mapping-frameworks, I think it might be interesting to think about MapStruct as the default in the meantime. |
@maybeec MapStruct is not a drop-in replacement. It is a code-generater and can not serve with a generic API to map arbitrary objects. IMHO the philosophy is so different that it is not really to be considered for our existing BeanMapper API. |
I created issue #147 for this and therefore close this issue as it has already been done as planned initially. |
For the record: |
The development of dozer has been restarted.
If we are not going for other mapping tools by default, we should at least migrate to the latest version of dozer fixing multiple often occurring issues in bigger projects.
https://github.com/DozerMapper/dozer
The text was updated successfully, but these errors were encountered: