You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/03 10:55:29 UTC

[GitHub] [airflow] potiuk commented on pull request #16267: Rename DAG concurrency settings for easier understanding

potiuk commented on PR #16267:
URL: https://github.com/apache/airflow/pull/16267#issuecomment-1203792584

   > Very clear name `max_active_tasks_per_dag` became obscured `max_active_tis_per_dag` somehow. How did it happen? This "tis" is not clear at all disappointed
   
   Your praised very simple `max_active_tasks_per_dag` was simply wrong name. There is a difference between task and task instance. The `tis` is often used all over the code for `task_instance` and this is right. `max_activet_tasks_instances_per_dag` would be simply too long. 
   
   It's all perfectly logical and accurate. As opposed to previous name which was wrong and illogical.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org