Skip to content

[SPARK-55288] Upgrade Netty to 4.2.9.Final#477

Closed
dongjoon-hyun wants to merge 1 commit intoapache:mainfrom
dongjoon-hyun:SPARK-55288
Closed

[SPARK-55288] Upgrade Netty to 4.2.9.Final#477
dongjoon-hyun wants to merge 1 commit intoapache:mainfrom
dongjoon-hyun:SPARK-55288

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Jan 30, 2026 •
edited
Loading

What changes were proposed in this pull request?

This PR aims to upgrade Netty to 4.2.9.Final by overriding the transitive one from the Apache Spark 4.1.x.

Why are the changes needed?

To bring the latest bug fixed version in Apache Spark K8s Operator like the Apache Spark 4.2.x.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

BEFORE

$ gradle spark-operator:dependencyInsight --configuration compileClasspath --dependency io.netty | grep '^io.netty' | awk -F: '{print $NF}' | sort | uniq -c
   2 4.1.119.Final -> 4.2.7.Final
   9 4.1.130.Final -> 4.2.7.Final
  50 4.2.7.Final

AFTER

$ gradle spark-operator:dependencyInsight --configuration compileClasspath --dependency io.netty | grep '^io.netty' | awk -F: '{print $NF}' | sort | uniq -c
   2 4.1.119.Final -> 4.2.9.Final
   9 4.1.130.Final -> 4.2.9.Final
   3 4.2.7.Final -> 4.2.9.Final
  22 4.2.9.Final
  25 4.2.9.Final (selected by rule)

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

Yes (Opus 4.5 on Claude Code v2.1.5)

@github-actions github-actions bot added the BUILD label Jan 30, 2026
@dongjoon-hyun dongjoon-hyun force-pushed the SPARK-55288 branch 2 times, most recently from cf683cd to d1feb50 Compare January 30, 2026 11:17
@dongjoon-hyun dongjoon-hyun marked this pull request as draft January 30, 2026 11:18
@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review January 30, 2026 11:25
@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Jan 30, 2026

Could you review this dependency PR (security and stability fix), @peter-toth ?

peter-toth
peter-toth approved these changes Jan 30, 2026
View reviewed changes
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.

Looks good, thanks @dongjoon-hyun.

@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Jan 30, 2026

Thank you always, @peter-toth !

Merged to main.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-55288 branch February 3, 2026 10:21
@dongjoon-hyun dongjoon-hyun added this to the 0.8.0 milestone Feb 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@peter-toth peter-toth peter-toth approved these changes

Assignees

No one assigned

Labels

BUILD

Projects

None yet

Milestone

0.8.0

Development

Successfully merging this pull request may close these issues.

2 participants

@dongjoon-hyun @peter-toth