[timeseries] Adding timeseries language endpoint and UI integration#16424
[timeseries] Adding timeseries language endpoint and UI integration#16424ankitsultana merged 3 commits intoapache:masterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #16424 +/- ##
============================================
+ Coverage 62.90% 63.18% +0.27%
+ Complexity 1386 1363 -23
============================================
Files 2867 3004 +137
Lines 163354 174091 +10737
Branches 24952 26641 +1689
============================================
+ Hits 102755 109993 +7238
- Misses 52847 55709 +2862
- Partials 7752 8389 +637
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
...src/main/java/org/apache/pinot/broker/api/resources/PinotBrokerTimeSeriesConfigResource.java
Outdated
Show resolved
Hide resolved
|
The failing test Root caused it and looks like it happened due to #16360 after adding in the aforementioned integration test. Removing this line fixes the test. |
|
I am checking the failure. I did repro on my laptop. Surprised - since all tests passed in my PR. |
|
I know the root cause. I'll work on a fix and open a PR. |
|
Draft PR: #16439 |
41ae3ef to
1a2be4a
Compare
1a2be4a to
c0c16af
Compare
Summary
This PR introduces support for retrieving and displaying enabled time series query languages configured at the broker level as a part of #16287.
/timeseries/languagesAPI to fetch languages from controller configpinot.timeseries.languages. (the same timeseries configs as broker and server).TimeseriesQueryPageReact component to dynamically fetch and display supported timeseries query languages:getTimeSeriesLanguages()torequests/index.tsfor UI to call the new controller endpoint.Since the response from the broker can be variable, the UI components are not hidden but disabled for editing.
Testing
Verified for both cases where timeseries is enabled and not enabled.
Timeseries Enabled UI
Timeseries Disabled UI