-
Notifications
You must be signed in to change notification settings - Fork 1.6k
POC (testing) -- test with Parquet Metadata Cache enabled #16988
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
Conversation
🤖 |
🤖: Benchmark completed Details
|
🤖 |
🤖: Benchmark completed Details
|
These are some pretty consistent and good improvements. Love it. Let's focus on getting
|
@alamb Would it make sense to have a benchmark to test small/point queries? I would be happy to contribute in case it makes sense. |
I think it would make sense if it covered something different than ClickBench
In terms of actual single row lookups, I am not sure about that benchmark it might end up being pretty specialized and hard to manage (the variability of benchmarks is already a problem for us). -- interestingly @zhuqi-lucas @JigaoLui and I were just discussing this earlier today: #17010 (comment) |
This PR is to test the performance of this PR from @nuno-faria
When the parquet metadata cache is enabled
I do not intend to merge this PR