You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2022/11/09 12:21:11 UTC
[GitHub] [calcite-avatica] snuyanzin opened a new pull request, #195: [CALCITE-5374] Upgrade jackson to 2.14.0
snuyanzin opened a new pull request, #195:
URL: https://github.com/apache/calcite-avatica/pull/195
The PR upgrading jackson to 2.14.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] F21 commented on pull request #195: [CALCITE-5374] Upgrade jackson to 2.14.0
Posted by GitBox <gi...@apache.org>.
F21 commented on PR #195:
URL: https://github.com/apache/calcite-avatica/pull/195#issuecomment-1309440018
@snuyanzin Can you please take a look at the failing tests for JDK8?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] snuyanzin closed pull request #195: [CALCITE-5374] Upgrade jackson from 2.10.0 to 2.14.1
Posted by GitBox <gi...@apache.org>.
snuyanzin closed pull request #195: [CALCITE-5374] Upgrade jackson from 2.10.0 to 2.14.1
URL: https://github.com/apache/calcite-avatica/pull/195
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] snuyanzin commented on pull request #195: [CALCITE-5374] Upgrade jackson to 2.14.0
Posted by GitBox <gi...@apache.org>.
snuyanzin commented on PR #195:
URL: https://github.com/apache/calcite-avatica/pull/195#issuecomment-1312563753
It seems the reason is that in 2.14.0 `com.fasterxml.jackson.databind.node.ObjectNode#with` became deprecated https://github.com/FasterXML/jackson-databind/issues/3568
So first need to update it in Calcite and replace deprecated methods and then could be updated for Avatica.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] julianhyde commented on pull request #195: [CALCITE-5374] Upgrade jackson to 2.14.0
Posted by GitBox <gi...@apache.org>.
julianhyde commented on PR #195:
URL: https://github.com/apache/calcite-avatica/pull/195#issuecomment-1329633534
Now I've merged the fix into Calcite, the Avatica JDK 8 test should pass. Once it passes let's merge this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] snuyanzin commented on pull request #195: [CALCITE-5374] Upgrade jackson to 2.14.0
Posted by GitBox <gi...@apache.org>.
snuyanzin commented on PR #195:
URL: https://github.com/apache/calcite-avatica/pull/195#issuecomment-1310671904
yes, sure will try find the reason
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica] snuyanzin commented on pull request #195: [CALCITE-5374] Upgrade jackson to 2.14.0
Posted by GitBox <gi...@apache.org>.
snuyanzin commented on PR #195:
URL: https://github.com/apache/calcite-avatica/pull/195#issuecomment-1329641680
Thanks for merging.
If you don't mind i would update it to 2.14.1 to include fix for [ObjectMapper default heap consumption increased significantly from 2.13.x to 2.14.0 ](https://github.com/FasterXML/jackson-databind/issues/3665)
I will try to upgrade both Calcite and Avatica today
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org