Skip to content

[SPARK-58861][BUILD] Upgrade Jackson to 2.22.1 - #58116

Closed
madsager-db wants to merge 2 commits into
apache:masterfrom
madsager-db:SPARK-58861
Closed

[SPARK-58861][BUILD] Upgrade Jackson to 2.22.1#58116
madsager-db wants to merge 2 commits into
apache:masterfrom
madsager-db:SPARK-58861

Conversation

@madsager-db

@madsager-dbmadsager-db commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR upgrades FasterXML Jackson from 2.22.0 to 2.22.1.

Why are the changes needed?

Jackson 2.22.1 is the latest patch release on the 2.22 line and addresses SPARK-58861.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI checks.

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

Generated-by: Codex GPT-5

@pan3793

Copy link
Copy Markdown
Member

Given our policy of JVM dependency management - only allow upgrading on major version (security issue may be evaluated case by case), and the versioning policy of Jackson, maybe we should only pick the LTS Jackson to use https://github.com/FasterXML/jackson

cc @dongjoon-hyun@LuciferYang@HyukjinKwon

@uros-buros-b left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also leaving a note here (just for reference) that this was previously attempted in #57982.

@madsager-db

Copy link
Copy Markdown
ContributorAuthor

Given our policy of JVM dependency management - only allow upgrading on major version (security issue may be evaluated case by case), and the versioning policy of Jackson, maybe we should only pick the LTS Jackson to use https://github.com/FasterXML/jackson

cc @dongjoon-hyun@LuciferYang@HyukjinKwon

I'm targeting the master branch here which I believe means that this is next version (Spark 5.0) only?

Also, the patch version update is to the version we are already using. We are already on 2.22.0 which looks like it is not LTS, and this is just updating to the latest patch version to address security vulnerabilities in that minor version. So, the only changes this is bringing in is this: https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.22.1.

@madsager-db

Copy link
Copy Markdown
ContributorAuthor

Also leaving a note here (just for reference) that this was previously attempted in #57982.

I wasn't aware of that one. It seems to do multiple libraries in one and has not landed yet. Maybe we can land this one to get these security fixes in and then work on the remainder in follow-ups?

@uros-buros-b left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, bump looks good to me, I won't merge yet until we all align
adding @HyukjinKwon and @dongjoon-hyun here to discuss more

@HyukjinKwonHyukjinKwon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

to me im fine

@HyukjinKwon

Copy link
Copy Markdown
Member

Merged to master.

@HyukjinKwon

Copy link
Copy Markdown
Member

Merge Summary:

Posted by merge_spark_pr.py

Sign up for freeto 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.

4 participants

@madsager-db@pan3793@HyukjinKwon@uros-b