potiuk commented on PR #36504: URL: https://github.com/apache/airflow/pull/36504#issuecomment-1872889631
I expect a long time to go/test all cases . But I love it :). But yeah, if we limit it for Connections and Variables for now, that would be cool. I think one general comment here is that not everyone will be able to use those drivers that support async operations I am afraid. There might be various limitations - dependencies, some specific corporate/managed DB configuration that is not supported - so, almost-for-sure - we should have a way to a) handle sync access b) fail async access (from triggers) when async access is done and the driver does not support it (?) c) possibly provide a way for Trigger developers to fallback to sync access (following #36492) I **think** that cleanest solution we should explicitly recognise sync/async access to Connections/Variables and enable them by a configuration flag for example and have a method/way to check it from Trigger. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
