Skip to content

Fix typo in 42953.significant.rst (max_active_runs -> max_active_tasks)#48472

Merged
kaxil merged 1 commit intoapache:mainfrom
TJaniF:patch-2
Mar 28, 2025
Merged

Fix typo in 42953.significant.rst (max_active_runs -> max_active_tasks)#48472
kaxil merged 1 commit intoapache:mainfrom
TJaniF:patch-2

Conversation

@TJaniF
Copy link
Contributor

@TJaniF TJaniF commented Mar 28, 2025

Noticed a typo in the newsfragment

the PR says:

Pursuant to mailing list vote, this PR changes meaning of DAG.max_active_tasks from "per dag" to "per dag run"

max_active_runs is the parameter limiting concurrent dag runs.

Related question for @dstandish: Was the AIRFLOW__CORE__MAX_ACTIVE_TASKS_PER_DAG config changed as well? I don't think from reading #42953 , but I might be missing another PR, just asking because
he draft docs still say the config is per dag, not per dag run and I think there should be parity between the config and the dag-level param if possible :)

The setting changed from across all dags to across an individual run in the PR is max_active_tasks, not max_active_runs, which is the dag parameter limiting number of concurrent dag runs.
@phanikumv phanikumv requested a review from dstandish March 28, 2025 06:08
@kaxil kaxil merged commit fc833fc into apache:main Mar 28, 2025
34 checks passed
pankajkoti pushed a commit to astronomer/airflow that referenced this pull request Mar 28, 2025
…s) (apache#48472)

The setting changed from across all dags to across an individual run in the PR is max_active_tasks, not max_active_runs, which is the dag parameter limiting number of concurrent dag runs.
shubham-pyc pushed a commit to shubham-pyc/airflow that referenced this pull request Apr 2, 2025
…s) (apache#48472)

The setting changed from across all dags to across an individual run in the PR is max_active_tasks, not max_active_runs, which is the dag parameter limiting number of concurrent dag runs.
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
…s) (apache#48472)

The setting changed from across all dags to across an individual run in the PR is max_active_tasks, not max_active_runs, which is the dag parameter limiting number of concurrent dag runs.
simonprydden pushed a commit to simonprydden/airflow that referenced this pull request Apr 8, 2025
…s) (apache#48472)

The setting changed from across all dags to across an individual run in the PR is max_active_tasks, not max_active_runs, which is the dag parameter limiting number of concurrent dag runs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants