Skip to content

[SPARK-56318][BUILD][4.1] Upgrade jackson to 2.21.2#55135

Open
manuzhang wants to merge 3 commits intoapache:branch-4.1from
manuzhang:upgrade-jackson-spark4.1
Open

[SPARK-56318][BUILD][4.1] Upgrade jackson to 2.21.2#55135
manuzhang wants to merge 3 commits intoapache:branch-4.1from
manuzhang:upgrade-jackson-spark4.1

Conversation

@manuzhang
Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Fixes vulnerability GHSA-72hv-8253-57qq

Why are the changes needed?

jackson-core 2.20.0 is affected.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing tests.

Was this patch authored or co-authored using generative AI tooling?

No.

@manuzhang
Copy link
Copy Markdown
Member Author

@dongjoon-hyun @pan3793 Please help take a look, thanks!

@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 2, 2026

We generally don't upgrade minor versions of dependencies in the release branches. Is it possible to have a patched Jackson 2.20.x? Or is Jackson 2.21 fully compatible(e.g., no default behavior change, no removal of deprecated API) with 2.20?

cc @pjfanning, could you provide some info?

@manuzhang
Copy link
Copy Markdown
Member Author

@pan3793 Looking at the release notes, there is only one patch between 2.20.0 and 2.21. Also, 2.21.1 is the patched version for affected versions >= 2.19.0, < 2.21.1.

@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 2, 2026

@manuzhang Jackson has a dozen code repos ... the CI failure is likely caused by jackson-module-scala pulling a new Scala version.

@manuzhang manuzhang force-pushed the upgrade-jackson-spark4.1 branch from 49bab5c to 655b338 Compare April 2, 2026 06:59
@manuzhang manuzhang force-pushed the upgrade-jackson-spark4.1 branch from 655b338 to 9bbb1db Compare April 2, 2026 10:14
@pjfanning
Copy link
Copy Markdown
Member

Jackson 2.21 is LTS while 2.20 is not. 2.21.3 has been released.

@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 2, 2026

Jackson 2.21 is LTS while 2.20 is not.

Alright, I used to think that Jackson's minor versions have the same support policy ...

cc @holdenk @dongjoon-hyun, do we want to accept such an upgrade for branch-4.1

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.

3 participants