shahar1 commented on PR #71337: URL: https://github.com/apache/airflow/pull/71337#issuecomment-6236213080
Thanks for working on this. With the default `BackgroundThreadTransport`, which is what the `stackdriver://` wiring, `from_config` and the deprecated alias all use, `send()` only puts the entry on an in-memory queue. The Cloud Logging API call happens in the library's background worker, which already catches and logs commit errors (`_safely_commit_batch`), so the new `except` branch can't run in supported configurations. It would only fire if a custom Python logging config passed `SyncTransport` explicitly; `remote_task_handler_kwargs` can't select one. The reporter's stack trace was never provided, and an earlier PR for the same fix (#68250) was closed for the same reason, so I'm closing this one. If you can reproduce the crash with the default transport, please open an issue with the traceback and we'll look at the real failure path. --- Drafted-by: Claude Code (Opus 5.5); reviewed by @shahar1 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]
