Skip to content

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Oct 21, 2025

What changes were proposed in this pull request?

Jackson 2.20 was released on August 28, 2025.

Release Notes: https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.20
Release Blog: https://cowtowncoder.medium.com/jackson-2-20-0-released-0cc58ed1ea9f

Why are the changes needed?

Bump Jackson to the latest 2.x version, Jackson 3.0.0 has been released in October, 2.20 probably is the last feature version of 2.x serials?

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GHA.

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

No.

@github-actions github-actions bot added the BUILD label Oct 21, 2025
Copy link
Contributor

@peter-toth peter-toth left a comment

Choose a reason for hiding this comment

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

LGTM, pending CI.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM (Pending CIs). Thank you, @pan3793 and Peter.

@vrozov
Copy link
Member

vrozov commented Oct 21, 2025

It will be good to test these change with Kinesis ASL. Upgrade to incompatible version of Jackson is what caused https://issues.apache.org/jira/browse/SPARK-53927 on 4.x

@pan3793
Copy link
Member Author

pan3793 commented Oct 22, 2025

@vrozov I don't have an AWS account to access Kinesis, is it possible to test with something like localstack? or API compatible service, e.g. MinIO for S3

@vrozov
Copy link
Member

vrozov commented Oct 22, 2025

@pan3793 IDK if something like MinIO is an option. I used actual Kinesis service to test https://issues.apache.org/jira/browse/SPARK-53927. I can try to test your PR using my Kinesis tomorrow.

Copy link
Member

@sarutak sarutak left a comment

Choose a reason for hiding this comment

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

@pan3793
I confirmed Kinesis tests pass with this change on my dev environment.
I'll merge this into master.
Thank you all!

@sarutak sarutak closed this in 3109488 Oct 22, 2025
@pan3793
Copy link
Member Author

pan3793 commented Oct 22, 2025

@sarutak, thank you for checking Kinesis tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants