Remove artificial time series chart delay (#3696)

* update timeseries-data-store to return data as long as the primary or primaryTotal data is available (rather than both) * assign totals query a higher priority
rilldata · Dec 14, 2023 · 135177d · 135177d · github-actions · Dec 14, 2023
1 parent 8bb5062
commit 135177d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/web-common/src/features/dashboards/time-series/timeseries-data-store.ts b/web-common/src/features/dashboards/time-series/timeseries-data-store.ts
@@ -159,16 +159,6 @@ export function createTimeSeriesDataStore(ctx: StateManagers) {
         ]) => {
           let timeSeriesData = primary?.data?.data;
 
-          if (!primary?.data || !primaryTotal?.data || !unfilteredTotal?.data) {
-            return {
-              isFetching:
-                metricsView.isFetching ||
-                primary?.isFetching ||
-                primaryTotal?.isFetching ||
-                unfilteredTotal?.isFetching,
-            };
-          }
-
           if (!primary.isFetching && interval) {
             timeSeriesData = prepareTimeSeries(
               primary?.data?.data,
@@ -178,7 +168,7 @@ export function createTimeSeriesDataStore(ctx: StateManagers) {
             );
           }
           return {
-            isFetching: primaryTotal?.isFetching || metricsView?.isFetching,
+            isFetching: !primary?.data && !primaryTotal?.data,
             isError: false, // FIXME Handle errors
             timeSeriesData,
             total: primaryTotal?.data?.data?.[0],

diff --git a/web-common/src/runtime-client/http-request-queue/priorities.ts b/web-common/src/runtime-client/http-request-queue/priorities.ts
@@ -16,6 +16,7 @@ export const QueryPriorities = {
   topk: 10,
   "rug-histogram": 10,
   "descriptive-statistics": 10,
+  totals: 30,
 };
 
 export function getPriority(type: string): number {