ephraimbuddy commented on PR #30165: URL: https://github.com/apache/airflow/pull/30165#issuecomment-1473506676
> Revert sounds like the right choice to me. > > I think it could be nice to add the test cases from: #30146 to prevent similarly bad changes being introduced in the future. But maybe that is better done in a followup PR. Feel free to make the PR once this is merged. Also we can add a comment on the signal handler itself to say that we don't handle callbacks when tasks are marked failed externally -- 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]
