Skip to content

transport: remove grpchttp2 package#8826

Merged
eshitachandwani merged 1 commit into
grpc:masterfrom
eshitachandwani:remove_newframer
Jan 13, 2026
Merged

transport: remove grpchttp2 package#8826
eshitachandwani merged 1 commit into
grpc:masterfrom
eshitachandwani:remove_newframer

Conversation

@eshitachandwani

Copy link
Copy Markdown
Member

This PR remove the grpchttp2 package since we no longer want to implement a custom http2 framer in grpc.

We originally planned a custom HTTP/2 framer to get rid of internal copies. However, we have updated gRPC to control data frame I/O (#8667, #8657) and will have changes that removes the bufio.Reader copy. This removes the need for a custom framer.

RELEASE NOTES: None

@eshitachandwani eshitachandwani added this to the 1.79 Release milestone Jan 13, 2026
@eshitachandwani eshitachandwani added Type: Internal Cleanup Refactors, etc Area: Transport Includes HTTP/2 client/server and HTTP server handler transports and advanced transport features. labels Jan 13, 2026
@codecov

codecov Bot commented Jan 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.34%. Comparing base (629ef39) to head (09088ac).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8826      +/-   ##
==========================================
+ Coverage   83.22%   83.34%   +0.12%     
==========================================
  Files         417      414       -3     
  Lines       32920    32753     -167     
==========================================
- Hits        27397    27298      -99     
+ Misses       4108     4061      -47     
+ Partials     1415     1394      -21     

see 24 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eshitachandwani eshitachandwani merged commit 25dbb81 into grpc:master Jan 13, 2026
15 of 17 checks passed
atollena pushed a commit to zhiyanfoo/grpc-go that referenced this pull request Jan 15, 2026
This PR remove the `grpchttp2` package since we no longer want to
implement a custom http2 framer in grpc.

We originally planned a custom HTTP/2 framer to get rid of internal
copies. However, we have updated gRPC to control data frame I/O
([grpc#8667](grpc#8667),
[grpc#8657](grpc#8657)) and will have
changes that removes the bufio.Reader copy. This removes the need for a
custom framer.

RELEASE NOTES: None
mbissa pushed a commit to mbissa/grpc-go that referenced this pull request Feb 16, 2026
This PR remove the `grpchttp2` package since we no longer want to
implement a custom http2 framer in grpc.

We originally planned a custom HTTP/2 framer to get rid of internal
copies. However, we have updated gRPC to control data frame I/O
([grpc#8667](grpc#8667),
[grpc#8657](grpc#8657)) and will have
changes that removes the bufio.Reader copy. This removes the need for a
custom framer.

RELEASE NOTES: None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Transport Includes HTTP/2 client/server and HTTP server handler transports and advanced transport features. Type: Internal Cleanup Refactors, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants