Nataneljpwd commented on PR #53492:
URL: https://github.com/apache/airflow/pull/53492#issuecomment-3176810402

   > Hi in your calculation of `tis_per_dag_count` you are partitioning by dag 
and task but i think maybe it's supposed to be just dag?
   > 
   > er... oh boy... what's the difference between `max_active_tis_per_dagrun` 
and `max_active_tasks` 
   > 
   > ok i guess `max_active_tis_per_dagrun` is per task across all dag runs.... 
that's a pain -- we should maybe deprecate that
   
   @dstandish 
   
   We know it is a little confusing, sometimes we still mix them up, there are 
discussions in the PRs of the features, Max active tasks is the maximum active 
tasks across a DAG across all Dagruns (globally, there is also a per dag 
setting) and Max active tis per dag is for mapped tasks, a better name is max 
active (mapped) tis per dag per dagrun, @Asquator even asked the PR author 
somewhere but I couldn't find it.


-- 
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]

Reply via email to