8000 Parquet: Add parquet queryable by jesusvazquez · Pull Request #11270 · grafana/mimir · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Parquet: Add parquet queryable #11270

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
Apr 24, 2025
Merged

Parquet: Add parquet queryable #11270

merged 14 commits into from
Apr 24, 2025

Conversation

jesusvazquez
Copy link
Contributor

Like we did with the parquet-converter but this time with the queryable.

Still WIP

@jesusvazquez jesusvazquez force-pushed the jvp/parquet-queryable branch from 925379b to d832ede Compare April 21, 2025 16:24
npazosmendez
npazosmendez reviewed Apr 21, 2025
View reviewed changes
npazosmendez
npazosmendez reviewed Apr 21, 2025
View reviewed changes
pkg/querier/parquet_queryable.go

minT, maxT, _ := q.minT, q.maxT, int64(0)

// TODO: honour limit hint
Copy link
Contributor

Choose a reason for hiding this comment

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

Bringing visibility

npazosmendez
npazosmendez reviewed Apr 21, 2025
View reviewed changes
pkg/querier/parquet_queryable.go

minT, maxT, _ := q.minT, q.maxT, int64(0)

// TODO: honour limit hint
Copy link
Contributor

Choose a reason for hiding this comment

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

Bringing visibility

@jesusvazquez jesusvazquez force-pushed the jvp/parquet-queryable branch from 710f443 to c62ebf0 Compare April 24, 2025 10:01
@jesusvazquez jesusvazquez marked this pull request as ready for review April 24, 2025 10:01
@jesusvazquez jesusvazquez requested review from stevesg and a team as code owners April 24, 2025 10:01
@jesusvazquez jesusvazquez merged commit 316780a into parquet-mimir Apr 24, 2025
7 checks passed
@jesusvazquez jesusvazquez deleted the jvp/parquet-queryable branch April 24, 2025 10:01
jesusvazquez added a commit that referenced this pull request May 2, 2025
@jesusvazquez
Parquet: Add parquet queryable (#11270)
056c7db
francoposa pushed a commit that referenced this pull request May 7, 2025
@jesusvazquez @francoposa
Parquet: Add parquet queryable (#11270)
1dde5fa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@npazosmendez npazosmendez npazosmendez left review comments

@francoposa francoposa Awaiting requested review from francoposa

@stevesg stevesg Awaiting requested review from stevesg stevesg is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jesusvazquez @npazosmendez
0