moomindani commented on PR #71840: URL: https://github.com/apache/airflow/pull/71840#issuecomment-6039939379
Yes, happy to take it over. The new PR is #74406. Instead of the async twins of the sync helpers, it fills the `databricks_conn` cache through `get_async_connection(..., hook=self)` at the start of `_a_do_api_call`, so the existing sync helpers keep working from the trigger. The diff shrinks to 8 lines. @saitejabandaru-in is credited as co-author. It also raises `common-compat` to `>=1.17.0`, since the `hook=` argument first shipped there. Verified against a real Databricks workspace with a deferrable `DatabricksRunNowOperator` and a DB-backed connection, official images: | core | provider | result | |---|---|---| | 3.0.6 | 7.21.0 (PyPI) | failed, `You cannot use AsyncToSync in the same thread as an async event loop` | | 3.0.6 | this PR | success, trigger polled `RUNNING` → `TERMINATED/SUCCESS` | | 3.1.0 | this PR | success | | 3.2.2 | this PR | success | --- Drafted-by: Claude Code (Opus 5.5); reviewed by @moomindani 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]
