-
Notifications
You must be signed in to change notification settings - Fork 91
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
#326: commons-beanutils:1.9.4 -> commons-beanutils2:2.0.0 #327
base: master
Are you sure you want to change the base?
Conversation
It has to be mentioned that beanutils2 2.0.0 was not released by Apache. However, it looks like https://github.com/melloware/commons-beanutils2 has released it on MavenCentral. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
Are there any breaking changes between commons-beanutils and commons-beanutils2 that can impact us?
@leonate : I don't know. Probably not. I can't say for sure. |
Hi ! |
I've had a quick look at it. I haven't found a commons-beanutils alternative. I don't have time for further research. If you want to you can make a counterproposal as a new pull request. Object property access should be just as fast. The solution should allow DynaBean support to be added. |
see #326