dstandish commented on PR #55799: URL: https://github.com/apache/airflow/pull/55799#issuecomment-3304306487
@ashb re > > The reason is, the code to hit the API wraps the get_connection call with async_to_sync, which is forbidden in the asyncio event loop. > > Yes but not quite. > > The issue is itit wasn't first wrapped in sync_to_async (which is what the async_to_sync is meant to "bubble out" of > > Most triggers are fine and so correctly call it in the right way. The ones they don't were always at risk of blocking the event loop (but in practice works have not blocked it does more than a few ms if they did. _Most_ of the time) isn't that what i wrote just above? > sometimes get_connection (which goes to the database) might be called without wrapping in sync_to_async. -- 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]
