ashb commented on a change in pull request #7160: [AIRFLOW-1467] Dynamic
pooling via allowing tasks to use more than one pool slot (depending upon the
need)
URL: https://github.com/apache/airflow/pull/7160#discussion_r366306476
##########
File path: airflow/models/baseoperator.py
##########
@@ -178,6 +178,9 @@ class derived from this one results in the creation of a
task object,
:param pool: the slot pool this task should run in, slot pools are a
way to limit concurrency for certain tasks
:type pool: str
+ :param pool_capacity: the number of pool slots this task should use (>= 1)
Review comment:
Capacity is the overall size of the pool, and in the logs etc we talk about
slots ("Not scheduling since there are %s open slots in pool %s") so do you
think this would be better named as `pool_slots`
WDYT @tooptoop4 ?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
