Skip to content

Update parquet common + fix queryStoreAfter config for parquet querier #6799

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

Merged
merged 4 commits into from
Jun 6, 2025

Conversation

alanprot
Copy link
Member

@alanprot alanprot commented Jun 6, 2025

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • [NA] Documentation added
  • [NA] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: alanprot <[email protected]>
@alanprot alanprot marked this pull request as ready for review June 6, 2025 16:46
@dosubot dosubot bot added component/querier go Pull requests that update Go code sync-vendor labels Jun 6, 2025
@alanprot alanprot changed the title Update parquet common Update parquet common + fix queryStoreAfter config for parquet querier Jun 6, 2025
Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

… we filter out the right chunks

Signed-off-by: alanprot <[email protected]>
@alanprot alanprot force-pushed the update-parquet-common branch from 88994ff to ab0d824 Compare June 6, 2025 17:15
alanprot added 2 commits June 6, 2025 11:21
Signed-off-by: alanprot <[email protected]>
@alanprot alanprot force-pushed the update-parquet-common branch from 3c523e4 to aa665d7 Compare June 6, 2025 18:52
@yeya24 yeya24 merged commit cd7d881 into cortexproject:master Jun 6, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/querier go Pull requests that update Go code size/L sync-vendor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants