Skip to content

Conversation

LuciferYang
Copy link
Contributor

What changes were proposed in this pull request?

This pr aims to upgrade kryo-shaded from 4.0.2 to 4.0.3.

Why are the changes needed?

This version brings some bug fixes and performance improvements related to chunked encoding:

full changes as follows:

Does this PR introduce any user-facing change?

No

How was this patch tested?

  • Pass GitHub Actions

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

No

@LuciferYang LuciferYang marked this pull request as draft April 16, 2025 06:41
@LuciferYang
Copy link
Contributor Author

image

The failed tests are not related to the current PR. Similar failures have also been continuously occurring on branch-4.0 GA.

cc @yaooqinn

@LuciferYang LuciferYang marked this pull request as ready for review April 17, 2025 02:20
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.

LuciferYang added a commit that referenced this pull request Apr 17, 2025
### What changes were proposed in this pull request?
This pr aims to upgrade `kryo-shaded` from 4.0.2 to 4.0.3.

### Why are the changes needed?
This version brings some bug fixes and performance improvements related to chunked encoding:
- mproved filling InputChunked buffer: EsotericSoftware/kryo#651
- Support skipping input chunks after a buffer underflow EsotericSoftware/kryo#850
- Avoid flush repeatedly when has finished flushing EsotericSoftware/kryo#978

full changes as follows:

- EsotericSoftware/kryo@kryo-parent-4.0.2...kryo-parent-4.0.3

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
- Pass GitHub Actions

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

Closes #50603 from LuciferYang/SPARK-51806-4.0.

Authored-by: yangjie01 <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
@LuciferYang
Copy link
Contributor Author

Merged into branch-4.0. Thanks @yaooqinn and @dongjoon-hyun

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

Successfully merging this pull request may close these issues.

3 participants