dabla commented on issue #54350: URL: https://github.com/apache/airflow/issues/54350#issuecomment-3176689292
> > > > > looks like https://github.com/apache/airflow/blob/main/providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/msgraph.py#L252 this is being used for both sync and async > > > > > > > > > > > > The MSGraphOperator is purely async, all operaions are done through a triggerer, which is async, there are no sync operations involved (except the next method). > > > > > > > > > no the get connection called from sync i believe it should be async > > > > > > That must something new (or something has changed in Airflow) as this has always been the case > > I think from day one the triggers must implement everything in async way, before AF3 its allowable sync version function to write and execute, but with AF3 the interaction to task sdk from the trigger should be async always otherwise the same error as seen above will happen :) > > Alteast sync function should use sync_to_async. Weird, cause we are using the MSGraphAsyncOperator in Airflow 3 and never had this issue, I even started a new PR to allow using it through start_from_trigger directly and even there we didn’t encounter this issue. Will check it out tomorrow with above release cause we are using the newer version. -- 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]
