xBis7 commented on code in PR #54103:
URL: https://github.com/apache/airflow/pull/54103#discussion_r2485361359
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -121,12 +121,11 @@ core:
default: "32"
max_active_tasks_per_dag:
description: |
- The maximum number of task instances allowed to run concurrently in
each DAG. To calculate
- the number of tasks that is running concurrently for a DAG, add up the
number of running
- tasks for all DAG runs of the DAG. This is configurable at the DAG
level with ``max_active_tasks``,
+ The maximum number of task instances allowed to run concurrently in
each dag run.
+ This is also configurable per-dag with ``max_active_tasks``,
Review Comment:
`max_active_tasks_per_dag` has actually been changed by a previous patch to
be applied on a per dag_run base and not per dag. That patch neglected to
update the docs as well.
When I first made the changes for the current PR I thought that it was per
dag until I ran the test suite and realized that it's per dag_run.
I just updated the description to make it reflect how it currently works.
--
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]
