gopidesupavan commented on issue #54350:
URL: https://github.com/apache/airflow/issues/54350#issuecomment-3176673845

   > > > > 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.  
   
   


-- 
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

Reply via email to