Skip to content

Exclude jackson-bom from Renovate updates - #10

Merged
joker1007 merged 1 commit into
mainfrom
pin-jackson-in-renovate
Aug 27, 2026
Merged

joker1007 merged 1 commit into
mainfrom
pin-jackson-in-renovate

Conversation

@joker1007

Copy link
Copy Markdown
Contributor

Summary

Renovate's grouped PR #7 includes a jackson-bom 2.20.1 → 2.22.2 bump. That violates the constraint documented in gradle/libs.versions.toml: jackson must match the version bundled with Trino 476 (airbase 336 → jackson 2.20.1), because com.fasterxml.jackson.annotation.* is parent-first in Trino's plugin classloader and a newer jackson-databind fails with NoClassDefFoundError on deploy. CI cannot catch this — it only manifests when the plugin is loaded by a real Trino server, not in the in-process test harness.

This adds a packageRules entry disabling com.fasterxml.jackson:jackson-bom updates, with a description explaining that jackson moves in lockstep with the trino version.

After merging

Renovate will regenerate PR #7 without the jackson bump, leaving the ADBC 0.24.0 / AssertJ / Derby updates, which can then be merged as-is.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Hyh23au5JR5bBfXxg3MKeh

jackson must match the version bundled with Trino: jackson annotation
classes are parent-first in Trino's plugin classloader, so a mismatched
jackson-databind fails with NoClassDefFoundError on deploy (see the note
in gradle/libs.versions.toml). Renovate grouped a jackson-bom 2.22.2
bump into its java minor updates PR, which CI cannot catch because the
issue only appears when the plugin is loaded by a real Trino server.
Disable jackson-bom updates; bump it manually together with trino.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Hyh23au5JR5bBfXxg3MKeh
@joker1007
joker1007 merged commit fc0756c into main Aug 27, 2026
1 check passed
@joker1007
joker1007 deleted the pin-jackson-in-renovate branch August 27, 2026 10:51
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

Successfully merging this pull request may close these issues.

1 participant