ashb commented on code in PR #45008:
URL: https://github.com/apache/airflow/pull/45008#discussion_r1890932816
##########
airflow/executors/base_executor.py:
##########
@@ -171,7 +174,8 @@ def queue_command(
else:
self.log.error("could not queue task %s", task_instance.key)
- def queue_workload(self, workload: workloads.All) -> None:
+ @provide_session
Review Comment:
Since you are passing an explicit session in when this is called I would
rather we didn't use the `provide_session` decorator (I think it encourages
poor session "hygine" as a result)
--
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]
