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

Upgrade Jackson version to latest patch (or later minor version) #941

Closed
cowtowncoder opened this issue Dec 5, 2017 · 2 comments
Closed

Comments

@cowtowncoder
Copy link

This is more of a minor request for maintenance, not bug per se, but I noticed that the project has a dependency to rather old patch version of Jackson, 2.4.3.
Since newer versions typically contain many bug fixes, I would suggest dependency upgrade.
The safest option would be simply to use latest 2.4 patch, 2.4.6:

http://mvnrepository.com/artifact/com.fasterxml.jackson.core/jackson-databind

but it might make sense to consider an upgrade to a later minor version as well.
This could help with transitive dependencies as well since most frameworks these days rely on Jackson 2.8, 2.8.9 being the most "popular" patch version (and 2.8.10 latest)

@merlimat
Copy link
Contributor

merlimat commented Dec 5, 2017

Thanks for pointing out @cowtowncoder, coming from Jackson maintainer, we'll take the advice seriously! :)

@merlimat merlimat added this to the 1.22.0-incubating milestone Dec 5, 2017
@cowtowncoder
Copy link
Author

@merlimat No prob -- let me know if I can help. I have some vested interest here too, many co-workers are actually evaluating use of Pulsar, lots of excitement. So really wishing all the best with development, hoping to learn more myself.

merlimat added a commit to merlimat/pulsar that referenced this issue Jan 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants