jscheffl commented on code in PR #66878:
URL: https://github.com/apache/airflow/pull/66878#discussion_r3410150594
##########
airflow-core/src/airflow/jobs/scheduler_job_runner.py:
##########
@@ -476,26 +477,22 @@ def _debug_dump(self, signum: int, frame: FrameType |
None) -> None:
self.log.info("\n\t".join(map(repr, callstack)))
self.log.info("-" * 80)
- def _executable_task_instances_to_queued(self, max_tis: int, session:
Session) -> list[TI]:
+ def _acquire_pool_capacity(
+ self, max_tis: int, session: Session
Review Comment:
I _think_ we had the rule with session as kwarg mainly for the
`@provide_session` decorator which is not used here. So a session needs to be
provided by caller anyway. So it is rather a nit.
--
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]
