Skip to content

fix(deps): align Jackson modules with BOM - #988

Merged
dkropachev merged 1 commit into
scylladb:scylla-3.xfrom
dkropachev:fix/jackson-bom-alignment-3x
Aug 3, 2026
Merged

fix(deps): align Jackson modules with BOM#988
dkropachev merged 1 commit into
scylladb:scylla-3.xfrom
dkropachev:fix/jackson-bom-alignment-3x

Conversation

@dkropachev

@dkropachev dkropachev commented Aug 3, 2026

Copy link
Copy Markdown

Summary

  • import the FasterXML Jackson BOM at 2.18.9
  • align core, databind, annotations, JAX-RS, and module dependencies
  • remove individual Jackson dependency-management entries

This is the 3.x counterpart of #987.

Tests

  • mvn -pl driver-core,driver-extras,driver-examples -am -DskipTests package
  • mvn -pl driver-extras -am -Dtest=JacksonJsonCodecTest -Dsurefire.failIfNoSpecifiedTests=false test
  • mvn -pl driver-core enforcer:enforce -Drules=requireUpperBoundDeps -DskipTests
  • verified dependency trees for core, extras, and examples resolve Jackson modules to 2.18.9

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The Maven configuration imports the Jackson BOM using ${jackson.version}. It removes explicit dependency management entries for jackson-core, jackson-databind, jackson-annotations, and jackson-dataformat-yaml.

Possibly related issues

Possibly related PRs

  • scylladb/java-driver#987 — Both PRs change Jackson version management to use a BOM and remove explicit module versions.

Suggested reviewers: scylladb-promoter

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the Jackson BOM alignment change.
Description check ✅ Passed The description directly explains the Jackson BOM update, dependency alignment, removals, and verification steps.
✨ Finishing Touches 💡 1
⚔️ Resolve merge conflicts 💡
  • Resolve merge conflict in branch fix/jackson-bom-alignment-3x

Comment @coderabbitai help to get the list of available commands.

@dkropachev
dkropachev force-pushed the fix/jackson-bom-alignment-3x branch from 0e28dca to 6db306a Compare August 3, 2026 17:00
@dkropachev
dkropachev requested review from nikagra and removed request for scylladb-promoter August 3, 2026 17:10

@nikagra nikagra left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dkropachev
dkropachev merged commit a732d83 into scylladb:scylla-3.x Aug 3, 2026
12 of 14 checks passed
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.

2 participants