dstandish commented on PR #30152: URL: https://github.com/apache/airflow/pull/30152#issuecomment-1473183500
Yeah I'm fine with it.and yeah I have noticed this before. User could have wrapped with sync to async and still been fine. But yeah it should be async. And, wrapping with suppress should mean that even if they implemented, it will actually still run, and the error (awaiting None) will get swallowed. So I think we're ok and we're at actually not breaking at all.... -- 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]
