From 8633fa7a5d8749ccb0ac9418b95a1fd34c066ed4 Mon Sep 17 00:00:00 2001 From: Alex Langenfeld Date: Fri, 14 Jun 2024 15:42:21 -0500 Subject: [PATCH] [ui] block backfills page on main content (#22561) We were blocking on the instance health query but not the main content query, so add that. ## How I Tested These Changes --- .../packages/ui-core/src/instance/InstanceBackfills.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js_modules/dagster-ui/packages/ui-core/src/instance/InstanceBackfills.tsx b/js_modules/dagster-ui/packages/ui-core/src/instance/InstanceBackfills.tsx index e5f04a8b2e6ea..10aaac043e5c2 100644 --- a/js_modules/dagster-ui/packages/ui-core/src/instance/InstanceBackfills.tsx +++ b/js_modules/dagster-ui/packages/ui-core/src/instance/InstanceBackfills.tsx @@ -67,7 +67,6 @@ export const InstanceBackfills = () => { InstanceHealthForBackfillsQuery, InstanceHealthForBackfillsQueryVariables >(INSTANCE_HEALTH_FOR_BACKFILLS_QUERY); - useBlockTraceOnQueryResult(queryData, 'InstanceHealthForBackfillsQuery'); const [statusState, setStatusState] = useQueryPersistedState>({ encode: (vals) => ({status: vals.size ? Array.from(vals).join(',') : undefined}), @@ -105,6 +104,7 @@ export const InstanceBackfills = () => { ? result.partitionBackfillsOrError.results : [], }); + useBlockTraceOnQueryResult(queryResult, 'InstanceBackfillsQuery'); // this is the main page content const refreshState = useQueryRefreshAtInterval(queryResult, FIFTEEN_SECONDS); const {loading, data} = queryResult;