Skip to content

fix: Propagate query stats from quantile & topk queries - #13831

Merged
benclive merged 4 commits into
mainfrom
fix-stats-propagation-for-topk-and-quantile
Sep 17, 2024
Merged

fix: Propagate query stats from quantile & topk queries#13831
benclive merged 4 commits into
mainfrom
fix-stats-propagation-for-topk-and-quantile

Conversation

@benclive

@benclive benclive commented Aug 9, 2024

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

  • Fixes query stats propagation from queriers after sharding/splitting for two newer query paths: TopKSketchesResponse and QuantileSketchResponse.
  • This means the query-frontend will show correct stats for the work completed executing a query.
@benclive
benclive requested a review from a team as a code owner August 9, 2024 09:55
@benclive
benclive requested a review from cstyan August 9, 2024 10:31
@cstyan
cstyan force-pushed the fix-stats-propagation-for-topk-and-quantile branch from a644a40 to de68680 Compare September 16, 2024 17:18
@pull-request-size pull-request-size Bot added size/S and removed size/M labels Sep 16, 2024
Signed-off-by: Callum Styan <callumstyan@gmail.com>
@benclive
benclive merged commit 78b275b into main Sep 17, 2024
@benclive
benclive deleted the fix-stats-propagation-for-topk-and-quantile branch September 17, 2024 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants