avanish-garg commented on issue #72426: URL: https://github.com/apache/airflow/issues/72426#issuecomment-5507020596
Draft PR with an implementation: https://github.com/apache/airflow/pull/72424 (re-targeted from the previous #72379, closed to re-open via the web form so the template labels applied correctly). Went with an additive-only approach (new hook, `on_task_instance_failed` left unchanged) to avoid a breaking change — see the PR description for the reasoning. Happy to rework based on feedback. cc @henry3260 @guan404ming @dheerajturaga — I see each of you has prior commits in `taskinstance.py`'s failure-handling path, so flagging in case this area is useful for you to weigh in on. cc @potiuk as well, given the listener-registration surface this also touches (same as #36516). -- 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]
