dabla commented on issue #54350: URL: https://github.com/apache/airflow/issues/54350#issuecomment-3188253185
> > > Please note that the issue is in provider connection, previously there were issues in triggerer deadlock that being fixed. so the real fix now require in provider connection retrieval. if anyone of you willing to contribute please let me know i will assign this to you :) > > > I dont have bandwidth to work until nextweek. > > > > > > So, would that mean an asynchronous implementation of get_connection is needed, or am I misunderstanding? I’m a bit surprised that this change was introduced without apparent testing, as it seems like it could have a significant impact. > > it require async get_connection implementation in provider. if you looks at aws and google providers they all have async implementation for getting connection in triggerer. so similar way azure should have async implementation and thats the correct way, previously it was worked as its direct db call and while calling the get_connection the event loop was in block state , this is not good :) > > It just that now async means async now :) so at-least sync_to_async functions should be used if anyone dont want to implement separate async functions > > I would call it as provider issue this is not from airflow. Ok, I understand now, but that would imply a lot of providers will need to be updated, as those will suffer the same issue. -- 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]
