-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
GH-35279: [C++][Parquet] Tools: enhancement Parquet print stats #35262
GH-35279: [C++][Parquet] Tools: enhancement Parquet print stats #35262
Conversation
Create an issue for ci failed: #35266 |
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.
This looks good, although (while it is small) this isn't quite a MINOR PR. Could you open an issue and link that before we merge?
|
Done. By the way, how can patch be a minor change? |
Basically minor changes usually just fix spelling or small issues in documentation or comments. Anything that affects actual logic isn't considered minor. See: https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes |
Get it, will attention that next time :) |
Benchmark runs are scheduled for baseline = c48916a and contender = a4b2408. a4b2408 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
…apache#35262) ### Rationale for this change Enhancement printing value for parquet stats ### What changes are included in this PR? 1. Make DebugPrintStats for RecordReader better 2. Make Stats printer better ### Are these changes tested? No ### Are there any user-facing changes? No * Closes: apache#35279 Authored-by: mwish <maplewish117@gmail.com> Signed-off-by: Will Jones <willjones127@gmail.com>
…apache#35262) ### Rationale for this change Enhancement printing value for parquet stats ### What changes are included in this PR? 1. Make DebugPrintStats for RecordReader better 2. Make Stats printer better ### Are these changes tested? No ### Are there any user-facing changes? No * Closes: apache#35279 Authored-by: mwish <maplewish117@gmail.com> Signed-off-by: Will Jones <willjones127@gmail.com>
…apache#35262) ### Rationale for this change Enhancement printing value for parquet stats ### What changes are included in this PR? 1. Make DebugPrintStats for RecordReader better 2. Make Stats printer better ### Are these changes tested? No ### Are there any user-facing changes? No * Closes: apache#35279 Authored-by: mwish <maplewish117@gmail.com> Signed-off-by: Will Jones <willjones127@gmail.com>
Rationale for this change
Enhancement printing value for parquet stats
What changes are included in this PR?
Are these changes tested?
No
Are there any user-facing changes?
No