Skip to content

Conversation

@panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Apr 18, 2024

What changes were proposed in this pull request?

The pr aims to upgrade netty from 4.1.108.Final to 4.1.109.Final.

Why are the changes needed?

https://netty.io/news/2024/04/15/4-1-109-Final.html
This version has brought some bug fixes and improvements, such as:

  • Fix DefaultChannelId#asLongText NPE (#13971)
  • Rewrite ZstdDecoder to remove the need of allocate a huge byte[] internally (#13928)
  • Don't send a RST frame when closing the stream in a write future while processing inbound frames (#13973)

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA.

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

No.

@github-actions github-actions bot added the BUILD label Apr 18, 2024
@dongjoon-hyun
Copy link
Member

Could you file a JIRA, @panbingkun ?

@panbingkun panbingkun changed the title [WIP] test netty 4.1.109.Final [SPARK-47896][BUILD] Upgrade netty to 4.1.109.Final Apr 18, 2024
@panbingkun
Copy link
Contributor Author

Could you file a JIRA, @panbingkun ?

@dongjoon-hyun
done, thank you very much! ❤️

@panbingkun panbingkun marked this pull request as ready for review April 18, 2024 05:57
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. Thank you, @panbingkun .
Merged to master.

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.

2 participants