Skip to content

Conversation

@arjan-bal
Copy link
Contributor

@arjan-bal arjan-bal commented Oct 23, 2025

RELEASE NOTES:

  • mem:
    • Avoid clearing buffers which newly allocated.
    • Clear large buffers (> 1MB) before re-using.

@arjan-bal arjan-bal requested review from dfawley and easwars October 23, 2025 20:24
@arjan-bal arjan-bal added Type: Bug Area: Transport Includes HTTP/2 client/server and HTTP server handler transports and advanced transport features. labels Oct 23, 2025
@arjan-bal arjan-bal added this to the 1.77 Release milestone Oct 23, 2025
@codecov
Copy link

codecov bot commented Oct 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.05%. Comparing base (f448a97) to head (c237d60).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8670      +/-   ##
==========================================
+ Coverage   81.97%   83.05%   +1.08%     
==========================================
  Files         417      417              
  Lines       40788    32296    -8492     
==========================================
- Hits        33435    26823    -6612     
+ Misses       5991     4093    -1898     
- Partials     1362     1380      +18     
Files with missing lines Coverage Δ
mem/buffer_pool.go 100.00% <100.00%> (ø)

... and 363 files with indirect coverage changes

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

@dfawley dfawley assigned arjan-bal and unassigned easwars and dfawley Oct 27, 2025
@arjan-bal arjan-bal changed the title mem: Avoid cleaning new buffers and clear buffers from simpleBufferPools mem: Avoid clearing new buffers and clear buffers from simpleBufferPools Oct 28, 2025
@arjan-bal arjan-bal merged commit 8ab0c82 into grpc:master Oct 28, 2025
17 of 21 checks passed
@arjan-bal arjan-bal deleted the buffer-pool-clears branch October 28, 2025 05:33
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: Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants