Skip to content

Conversation

arjan-bal
Copy link
Contributor

This PR enables buffer pooling in the benchmark test to align with the library's current default configuration.

The benchmark originally disabled buffer pooling because the feature was opt-in when introduced (#5862). Since buffer pooling is now enabled by default (#7356), this change ensures the benchmark accurately measures the performance of gRPC's default behavior.

RELEASE NOTES: N/A

@arjan-bal arjan-bal added this to the 1.77 Release milestone Oct 8, 2025
@arjan-bal arjan-bal requested a review from easwars October 8, 2025 19:20
@arjan-bal arjan-bal added Type: Testing Area: Testing Includes tests and testing utilities that we have for unit and e2e tests within our repo. labels Oct 8, 2025
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.91%. Comparing base (ece7397) to head (7e43ea6).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8638      +/-   ##
==========================================
- Coverage   82.17%   81.91%   -0.27%     
==========================================
  Files         415      415              
  Lines       40709    40693      -16     
==========================================
- Hits        33453    33332     -121     
- Misses       5883     5989     +106     
+ Partials     1373     1372       -1     

see 27 files with indirect coverage changes

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

@easwars easwars assigned arjan-bal and unassigned easwars Oct 8, 2025
@arjan-bal arjan-bal merged commit 0e9ac5d into grpc:master Oct 9, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Testing Includes tests and testing utilities that we have for unit and e2e tests within our repo. Type: Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants