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/01/16 19:18:11 UTC

[GitHub] [airflow] ephraimbuddy commented on pull request #20391: Fix task instance concurrency limit in a DAG affecting other DAGs

ephraimbuddy commented on pull request #20391:
URL: https://github.com/apache/airflow/pull/20391#issuecomment-1013935448


   > I'm kinda torn on the name `last_scheduling_decision` here. By the name, one could assume it'd be populated any time the scheduler considers the TI, not just when `max_active_tasks` is hit. It makes me wonder if we should populate that in more places?
   > 
   > Related, nulling it out after another task finishes also feels odd given the name of the column.
   
   Makes sense, do you have suggestions on the name? Thinking about it too


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