Skip to content

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

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

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

Conversation

@dkropachev

Copy link
Copy Markdown

Summary

  • import the FasterXML Jackson BOM at 2.22.1
  • align core, databind, annotations, JAX-RS, and module dependencies
  • remove independent Jackson version pins

Tests

  • mvn -pl core -Dtest=JsonCodecTest,CloudConfigFactoryTest,InsightsClientTest,ExecutionProfilesInfoFinderTest test
  • mvn -pl core enforcer:enforce -Drules=requireUpperBoundDeps -DskipTests
  • mvn -pl core-shaded -am -DskipTests package
  • verified core and examples dependency trees resolve one Jackson release train

Closes #986

Copilot AI review requested due to automatic review settings August 3, 2026 16:37
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: QUIET

Plan: Pro Plus

Run ID: 678c99f9-dfae-4eaf-a924-02a8c2ac072c

📥 Commits

Reviewing files that changed from the base of the PR and between 3ced562 and 82cd8c7.

📒 Files selected for processing (2)
  • examples/pom.xml
  • pom.xml
💤 Files with no reviewable changes (1)
  • examples/pom.xml

📝 Walkthrough

Walkthrough

The root POM updates Jackson to 2.22.1, removes the separate jackson-databind.version property, and imports the Jackson BOM. The examples POM removes explicit versions from Jackson JAX-RS dependencies so Maven resolves them through dependency management.

Possibly related PRs

Suggested labels: dependencies, java

Suggested reviewers: copilot, scylladb-promoter

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the Jackson BOM alignment change.
Description check ✅ Passed The description directly covers the Jackson BOM, dependency alignment, version pin removal, tests, and linked issue.
Linked Issues check ✅ Passed The changes import Jackson BOM 2.22.1, remove separate version pins, and allow examples to inherit aligned versions as required by #986.
Out of Scope Changes check ✅ Passed All reported changes support Jackson dependency alignment and the requirements in issue #986.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitai coderabbitai Bot added dependencies Pull requests that update a dependency file java Pull requests that update Java code labels Aug 3, 2026

Copilot AI 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.

Pull request overview

Aligns all Jackson dependencies to the 2.22.1 release train using the official BOM, resolving issue #986.

Changes:

  • Imports jackson-bom 2.22.1 and removes independent version pins.
  • Lets example JAX-RS dependencies inherit BOM-managed versions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pom.xml Centralizes Jackson dependency management through the BOM.
examples/pom.xml Removes redundant JAX-RS version declarations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@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 be4b942 into scylladb:scylla-4.x Aug 3, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align Jackson dependencies with FasterXML BOM

3 participants