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 to measure multi-stage block serde cost #13336

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

gortiz
Copy link
Contributor

@gortiz gortiz commented Jun 7, 2024

This PR adds a benchmark that measures the cost of creating, serializing and deserializing multi-stage blocks.

This should be compared with results in #13303

@gortiz gortiz requested a review from Jackie-Jiang June 7, 2024 14:08
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.56%. Comparing base (59551e4) to head (02421fd).
Report is 566 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13336       +/-   ##
=============================================
- Coverage     61.75%   46.56%   -15.19%     
- Complexity      207     1096      +889     
=============================================
  Files          2436     1939      -497     
  Lines        133233   102921    -30312     
  Branches      20636    16557     -4079     
=============================================
- Hits          82274    47929    -34345     
- Misses        44911    51443     +6532     
+ Partials       6048     3549     -2499     
Flag Coverage Δ
custom-integration1 ?
integration ?
integration1 ?
integration2 ?
java-11 46.56% <ø> (-15.15%) ⬇️
java-21 ?
skip-bytebuffers-false 46.56% <ø> (-15.18%) ⬇️
skip-bytebuffers-true ?
temurin 46.56% <ø> (-15.19%) ⬇️
unittests 46.56% <ø> (-15.18%) ⬇️
unittests1 46.56% <ø> (-0.33%) ⬇️
unittests2 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Jackie-Jiang Jackie-Jiang merged commit dbeeaaf into apache:master Jun 7, 2024
19 of 20 checks passed
@gortiz gortiz deleted the multi-stage-serde-benchmark branch June 10, 2024 11:22
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