bbovenzi commented on code in PR #45977:
URL: https://github.com/apache/airflow/pull/45977#discussion_r1936110478
##########
airflow/ui/src/queries/usePatchDagRun.ts:
##########
@@ -45,7 +48,12 @@ export const usePatchDagRun = ({
const queryClient = useQueryClient();
const onSuccessFn = async () => {
- const queryKeys = [UseDagRunServiceGetDagRunKeyFn({ dagId, dagRunId }),
[useDagRunServiceGetDagRunsKey]];
+ const queryKeys = [
+ UseDagRunServiceGetDagRunKeyFn({ dagId, dagRunId }),
+ [useDagRunServiceGetDagRunsKey],
+ [useTaskInstanceServiceGetTaskInstancesKey, { dagId, dagRunId }],
Review Comment:
I have been running into this issue too with autorefresh. The real issue is
with how our query keys are autogenerated with an object as a second argument
by default. This doesn't allow us to easily compare query keys.
[Issue for
codgen](https://github.com/7nohe/openapi-react-query-codegen/issues/174)
We can specify our own array keys for each query. Since we know the order of
dagId, dagRunId, taskId is important. But that is a lot of manual work and
subject to mistakes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
