Skip to content

Conversation

@ejona86
Copy link
Member

@ejona86 ejona86 commented Aug 13, 2025

This implicitly disables NettyAdaptiveCumulator (#11284), which can have a performance impact. We delayed upgrading Netty to give time to rework the optimization, but we've gone too long already without upgrading which causes problems for vulnerability tracking.


The intention is to backport this to 1.75.x, so the README says it was upgraded in 1.75.x.

This implicitly disables NettyAdaptiveCumulator (grpc#11284), which can have a
performance impact. We delayed upgrading Netty to give time to rework
the optimization, but we've gone too long already without upgrading
which causes problems for vulnerability tracking.
@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Aug 13, 2025
@ejona86 ejona86 requested a review from sergiitk August 13, 2025 20:22
@ejona86
Copy link
Member Author

ejona86 commented Aug 13, 2025

CC @kannanjgithub

@sergiitk
Copy link
Member

ref cl/772178002

@ejona86 ejona86 merged commit 95d16d8 into grpc:master Aug 13, 2025
16 checks passed
@ejona86 ejona86 deleted the netty-4.1.124 branch August 13, 2025 21:23
kannanjgithub added a commit that referenced this pull request Aug 18, 2025
Backport of #12274 to v1.75.x.
---
This implicitly disables NettyAdaptiveCumulator (#11284), which can have
a performance impact. We delayed upgrading Netty to give time to rework
the optimization, but we've gone too long already without upgrading
which causes problems for vulnerability tracking.

-----

Co-authored-by: Eric Anderson <[email protected]>
@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label Aug 18, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants