MaksYermak commented on issue #49187: URL: https://github.com/apache/airflow/issues/49187#issuecomment-3136374023
@pankajkoti sorry my message was uncertain. I wanted to say that we can't remove `provide_session` because it's needed for the `get_task_instance()` function. The call for this function is under if-statement ([code](https://github.com/apache/airflow/blob/5edab62ca979fd19d125cce8891fc52680b99a80/providers/google/src/airflow/providers/google/cloud/triggers/dataproc.py#L174C1-L180C56)) and will be called only when users use Google provider with AF2. Maybe it makes sense to put this `from airflow.utils.session import provide_session` under if-statement and import it only for AF2 to not affect AF3. But in my opinion we shouldn't remove it completely, because it will break the code. -- 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]
