hterik commented on PR #35653: URL: https://github.com/apache/airflow/pull/35653#issuecomment-1812328332
> An in general need to check other Airflow codebase (core), that we do not miss to catch `AirflowTaskTimeout` where it required, because right now we could catch as `AirflowException` in some places Good point. I would need a lot more help from someone with deeper knowledge of Airflow internals to understand and review full scope of this. I found a few such issues in taskinstance.py, pushed in latest patchset. Question there is if we should loosen constraint on those types to just accept `BaseException` instead of listing all types in a Union. -- 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]
