Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

benchmark: wire in new gzip compressor #7486

Merged
merged 3 commits into from
Aug 13, 2024
Merged

Conversation

PapaCharlie
Copy link
Contributor

@PapaCharlie PapaCharlie commented Aug 6, 2024

Without this change, the benchmarks did not actually include executions with the new gzip compressor from the encoding/gzip package. This adds the option to run the suite with said new compressor and adds it to the "all" suite as well.

RELEASE NOTES: none

Without this change, the benchmarks did not actually include executions with
the new gzip compressor from the `encoding/gzip` package. This adds the option
to run the suite with said new compressor and adds it to the "all" suite as
well.
Copy link

codecov bot commented Aug 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.58%. Comparing base (e524655) to head (58ebe9a).
Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7486      +/-   ##
==========================================
+ Coverage   81.40%   81.58%   +0.18%     
==========================================
  Files         357      357              
  Lines       27243    27224      -19     
==========================================
+ Hits        22176    22211      +35     
+ Misses       3850     3809      -41     
+ Partials     1217     1204      -13     

see 26 files with indirect coverage changes

benchmark/benchmain/main.go Outdated Show resolved Hide resolved
@dfawley dfawley changed the title Wire in new gzip compressor in benchmark benchmark: wire in new gzip compressor Aug 13, 2024
@dfawley dfawley assigned easwars and unassigned PapaCharlie Aug 13, 2024
@easwars easwars merged commit 1008562 into grpc:master Aug 13, 2024
11 checks passed
infovivek2020 pushed a commit to infovivek2020/grpc-go that referenced this pull request Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants