-
Notifications
You must be signed in to change notification settings - Fork 901
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
Fixing parquet benchmarks #7214
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 👍
Codecov Report
@@ Coverage Diff @@
## branch-0.18 #7214 +/- ##
===============================================
+ Coverage 82.09% 82.20% +0.11%
===============================================
Files 97 98 +1
Lines 16474 16692 +218
===============================================
+ Hits 13524 13722 +198
- Misses 2950 2970 +20
Continue to review full report at Codecov.
|
@gpuciboth merge |
@gpucibot merge |
return_filemetadata
was removed in one of the recent PR, and missed to remove it in benchmarks.