-
Couldn't load subscription status.
- Fork 837
Support vertical sharding for parquet queryable #6879
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
Support vertical sharding for parquet queryable #6879
Conversation
1c79456 to
0c64f1b
Compare
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.
Amazing! thanks
|
The fuzz failure seems related. I will spend more time investigating it |
efb09e0 to
bc6e293
Compare
Signed-off-by: yeya24 <[email protected]>
Signed-off-by: yeya24 <[email protected]>
Signed-off-by: yeya24 <[email protected]>
bc6e293 to
a3bc1f0
Compare
Signed-off-by: yeya24 <[email protected]>
Signed-off-by: yeya24 <[email protected]>
Signed-off-by: yeya24 <[email protected]>
Signed-off-by: yeya24 <[email protected]>
|
Merging as correctness test is able to pass if disabling vertical sharding for binary expression without type of queries if parquet is enabled. |
What this PR does:
This PR implements vertical sharding for parquet queryable by leveraging a series label filtering hook in parquet-common. See prometheus-community/parquet-common#83. Since that PR is still not merged so mark this PR as draft.
Integration test for parquet querier has been updated to enable vertical sharding.
Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.mdupdated - the order of entries should be[CHANGE],[FEATURE],[ENHANCEMENT],[BUGFIX]