potiuk commented on PR #66571: URL: https://github.com/apache/airflow/pull/66571#issuecomment-4485761172
Addressed in 9c399930de: - @Lee-W — switched to `str(ti.id)` (3 places) and `exc_info=exc` for both `except` paths so the traceback is preserved. - @amoghrajesh — trimmed the past-tense sentence from the comment; also added `task-sdk/tests/task_sdk/test_log.py` with 5 tests for `upload_to_remote()` covering the three failure paths plus silent-no-path and silent-success. --- Drafted-by: Claude Code (Opus 4.7); reviewed by @potiuk before posting -- 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]
