GitHub user Asquator edited a comment on the discussion: why Dag's 
on_callback_failure have first taskinstance in dagrun?

I support the idea that dag callbacks should not receive any `TI` at all. It's 
weird and poorly defined. Maybe it should remain for back comp and that's what 
I did in the PR while improving the relevancy of the task passed. Anyway, it 
will be nice to add a `CallbackMeta` structure and inject it into the context 
to be able to differentiate between task callbacks and dag callbacks in case 
the same callback function is used in multiple places. I described the desired 
behavior here: https://github.com/apache/airflow/issues/61119

GitHub link: 
https://github.com/apache/airflow/discussions/24029#discussioncomment-15726307

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to