Skip to content

Conversation

@stoty
Copy link
Contributor

@stoty stoty commented Sep 10, 2025

No description provided.

@F21
Copy link
Member

F21 commented Sep 10, 2025

JDK 8 tests for Avatica main seem to be failing.

@stoty
Copy link
Contributor Author

stoty commented Sep 10, 2025

Yes.
Calcite uses Jackson 2.14, and it triggers deprecation warnings which fail the build.

@stoty
Copy link
Contributor Author

stoty commented Sep 10, 2025

We need to update Jackson in both or neither.

@stoty stoty changed the title [CALCITE-7171] Update Jackson from 2.15.4 to 2.18.4.1 and switch to using jackson-bom [CALCITE-7171] Update Jackson from 2.15.4 to 2.18.4.1 and switch to using jackson-bom in Avatica Sep 10, 2025
@rubenada
Copy link
Contributor

Calcite Jackson update has been merged (apache/calcite#4530); which should unblock the current PR

@stoty
Copy link
Contributor Author

stoty commented Sep 22, 2025

merged manually.

@stoty stoty closed this Sep 22, 2025
@stoty stoty reopened this Sep 22, 2025
@stoty
Copy link
Contributor Author

stoty commented Sep 22, 2025

there was no +1, force-pushed the previus version .

@stoty stoty requested review from F21, caicancai and rubenada September 22, 2025 04:47
@stoty stoty merged commit 1e05be2 into apache:main Sep 22, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants