-
Notifications
You must be signed in to change notification settings - Fork 819
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 fasterxml #2912
base: master
Are you sure you want to change the base?
Upgrade fasterxml #2912
Conversation
Hi, thanks for opening the PR. Could you clarify the reason for upgrading Note that we are currently pinning |
thanks @xiazcy for the reply I was't aware that we cannot upgrade to release candidates. I am committing again by upgrading to official released version. |
Hi and thanks for contributing. I was wondering what kind of testing you did for this PR. As noted earlier, it seems like this version was pinned for a reason. Have you looked into any potential issues with Spark/Scala if you change this Jackson version? |
It seems like progress on this PR has stalled and it isn't able to pass the smoke tests due to library version convergence issues. More work and testing will need to be done in order to move this through. Is anyone able to help with this? As this PR has been open for a while, if no one is able to help update it in the next week then I think we should close it for the time being until someone is able to work on it. |
No description provided.