Skip to content

[timeseries] Added query stats panel to timeseries UI#17423

Merged
ankitsultana merged 2 commits intoapache:masterfrom
shauryachats:timeseriesui_stats
Dec 29, 2025
Merged

[timeseries] Added query stats panel to timeseries UI#17423
ankitsultana merged 2 commits intoapache:masterfrom
shauryachats:timeseriesui_stats

Conversation

@shauryachats
Copy link
Copy Markdown
Collaborator

@shauryachats shauryachats commented Dec 23, 2025

Summary

This PR adds a stats panel to the Timeseries query UI and consolidates broker response processing logic between SQL and Timeseries query flows.

Since #17414 replaces broker compatible timeseries API to be leveraged for all timeseries flows, this PR also consolidates broker response processing logic into reusable utilizes for both SQL and timeseries query.

Testing

Successfully tested that the stats panel is visible correctly in both timeseries and SQL flow.

Screen.Recording.2025-12-23.at.7.05.10.PM.mov

Note: It is noticed that for timeseries queries, the timeUsedMs is showing to be 0, possibly an oversight in the implementation in #17170. It will be fixed in a subsequent PR.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.30%. Comparing base (851ac53) to head (1b80846).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17423      +/-   ##
============================================
+ Coverage     63.26%   63.30%   +0.03%     
  Complexity     1474     1474              
============================================
  Files          3154     3154              
  Lines        187940   187940              
  Branches      28771    28771              
============================================
+ Hits         118907   118969      +62     
+ Misses        59819    59749      -70     
- Partials       9214     9222       +8     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.27% <ø> (+0.02%) ⬆️
java-21 63.22% <ø> (-0.02%) ⬇️
temurin 63.30% <ø> (+0.03%) ⬆️
unittests 63.29% <ø> (+0.03%) ⬆️
unittests1 55.68% <ø> (-0.01%) ⬇️
unittests2 33.95% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ankitsultana ankitsultana merged commit cf71cc0 into apache:master Dec 29, 2025
18 checks passed
@ankitsultana ankitsultana added the timeseries-general-availability Work for making Timeseries Engine GA in Pinot (e.g. Controller UI support, debuggability, etc.) label Dec 29, 2025
@xiangfu0 xiangfu0 added timeseries-engine Tracking tag for generic time-series engine work ui UI related issue labels Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

timeseries-engine Tracking tag for generic time-series engine work timeseries-general-availability Work for making Timeseries Engine GA in Pinot (e.g. Controller UI support, debuggability, etc.) ui UI related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants