Skip to content

[improve][all] Upgraded Jackson to 2.21 LTS and fixed a few gradle assemble warnings#25504

Open
abhilashmandaliya wants to merge 1 commit intoapache:masterfrom
abhilashmandaliya:jackson-2.21-lts
Open

[improve][all] Upgraded Jackson to 2.21 LTS and fixed a few gradle assemble warnings#25504
abhilashmandaliya wants to merge 1 commit intoapache:masterfrom
abhilashmandaliya:jackson-2.21-lts

Conversation

@abhilashmandaliya
Copy link
Copy Markdown
Contributor

Main Issue: Upgraded Jackson to 2.21 LTS

Motivation

The currently used Jackson version 2.18.6 is vulnerable and older. Also, the user generating a new pulsar sink with a higher Jackson version encounters an error due to a signature change in a few Jackson classes.

Modifications

  • Upgraded all Jackson libraries to version 2.21.0 and jackson-annotations to 2.21 (that doesn't have a patch version anymore)

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Copy link
Copy Markdown
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

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

Looks good, just checkstyle issues reported in the CI job

@abhilashmandaliya abhilashmandaliya changed the title [improvement][all] Upgraded Jackson to 2.21 LTS and fixed a few gradle assemble warnings [improve][all] Upgraded Jackson to 2.21 LTS and fixed a few gradle assemble warnings Apr 11, 2026
# Conflicts:
#	pulsar-common/src/main/java/org/apache/pulsar/common/util/ObjectMapperFactory.java
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