-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[C++][Parquet] Enable size statistics by default #45227
Comments
What about page index (which not yet enabled by default)? Or do we enable SizeStatisticsLevel::ColumnChunk by default for now? |
The benchmark enables it when PageAndColumnChunk is selected, right? |
The benchmark enables page index in all cases. Should I change it to add a baseline with page index disabled? |
Ah, that could be informative indeed. |
Update: I just extend the size_stats_benchmark by adding a third variable to control page index
|
Describe the enhancement requested
Now that #45202 , the overhead of computing and writing out the size statistics seems sufficiently negligible that we should probably enable it by default. This may allow performance improvements in readers when they are updated to take advantage of the information.
Component(s)
C++, Parquet
The text was updated successfully, but these errors were encountered: