Skip to content
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

tsdb: Implement limit in block querier #14902

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

harry671003
Copy link
Contributor

@harry671003 harry671003 commented Sep 13, 2024

The storage.Querier not supports Limit param through hints. This PR uses the Limit from the hint passed to the block querier to return truncated results.

Related: #12795

@aknuds1
Copy link
Contributor

aknuds1 commented Sep 16, 2024

Tests are broken.

@harry671003 harry671003 force-pushed the implement_limit_in_block_querier branch from 3cfe2d1 to a808ccf Compare September 16, 2024 22:13
Signed-off-by: 🌲 Harry 🌊 John 🏔 <johrry@amazon.com>
@harry671003 harry671003 force-pushed the implement_limit_in_block_querier branch from a808ccf to 186fdab Compare September 16, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants