-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
ARROW-14025: [R][C++] PreBuffer is not enabled when scanning parquet via exec nodes #11386
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.
Looks correct to me. Let's run benchmarks real quick just to test.
@ursabot please benchmark lang=R |
Benchmark runs are scheduled for baseline = 157d48c and contender = c4ddad1. Results will be available as each benchmark for each run completes. |
Great. Which benchmark do you expect to be affected by this, and how much? |
Benchmark runs are scheduled for baseline = 71612e3 and contender = 0fbd5e4. 0fbd5e4 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Also includes an unrelated CSV dataset test, just to confirm that FragmentScanOptions are passed through correctly.