Skip to content

Commit 2cdbfd5

Browse files
petedmarshXirp Agent (gpt-6.1-sol)
andcommitted
netty: Increase HPACK encoder bucket count to 64
Increase the client and server HPACK encoder bucket counts from 16 to 64, matching the new Netty default. On newer Netty versions, 64 buckets allow encoder tables up to 4 KiB within the peer-advertised limit. Use 64 instead of 128 as requested in review: the adjusted benchmark shows comparable performance, while larger dynamic tables need further improvements to Netty’s encoder. Advertised table sizes and the Netty dependency version remain unchanged. Co-Authored-By: Codex <noreply@openai.com> Co-Authored-By: Xirp Agent (gpt-6.1-sol) <xirp-agent@users.noreply.ghe.spotify.net>
1 parent 4d0d548 commit 2cdbfd5

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

‎netty/src/main/java/io/grpc/netty/NettyClientHandler.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ static NettyClientHandler newHandler(
176176
Http2HeadersDecoder headersDecoder = new GrpcHttp2ClientHeadersDecoder(maxHeaderListSize);
177177
Http2FrameReader frameReader = new DefaultHttp2FrameReader(headersDecoder);
178178
Http2HeadersEncoder encoder = new DefaultHttp2HeadersEncoder(
179-
sensitivityDetector(neverIndexedMetadataKeys), false, 16, Integer.MAX_VALUE);
179+
sensitivityDetector(neverIndexedMetadataKeys), false, 64, Integer.MAX_VALUE);
180180
Http2FrameWriter frameWriter = new DefaultHttp2FrameWriter(encoder);
181181
Http2Connection connection = new DefaultHttp2Connection(false);
182182
UniformStreamByteDistributor dist = new UniformStreamByteDistributor(connection);

‎netty/src/main/java/io/grpc/netty/NettyServerHandler.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ static NettyServerHandler newHandler(
189189
Http2HeadersEncoder encoder = new DefaultHttp2HeadersEncoder(
190190
NettyClientHandler.sensitivityDetector(neverIndexedMetadataKeys),
191191
false,
192-
16,
192+
64,
193193
Integer.MAX_VALUE);
194194
Http2FrameWriter frameWriter =
195195
new Http2OutboundFrameLogger(new DefaultHttp2FrameWriter(encoder), frameLogger);

0 commit comments

Comments
 (0)