Gautam-Bharadwaj commented on PR #63664: URL: https://github.com/apache/airflow/pull/63664#issuecomment-4065733428
Thanks for the review Jarek! I've rebased the branch and added comprehensive unit tests for `timeout_with_traceback` in `airflow-core/tests/unit/utils/test_timeout_traceback.py`. The tests cover: - Normal behavior on Unix/Linux systems with `SIGALRM`. - Graceful handling of `AttributeError` (case when `SIGALRM` is missing on Windows). - Graceful handling of `ValueError` (case when called from a non-main thread). - Verified that a warning is logged when timeout is unsupported. - Verified that `TimeoutException` is correctly raised when the handler is triggered. Ready for review! 👋 -- 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]
