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 2019/04/27 03:03:07 UTC

[GitHub] [airflow] milton0825 commented on a change in pull request #5194: [WIP][AIRFLOW-4419] Refine concurrency check in scheduler

milton0825 commented on a change in pull request #5194: [WIP][AIRFLOW-4419] Refine concurrency check in scheduler
URL: https://github.com/apache/airflow/pull/5194#discussion_r279141113
 
 

 ##########
 File path: airflow/jobs.py
 ##########
 @@ -1014,9 +1013,10 @@ def _change_state_for_tis_without_dagrun(self,
             )
 
     @provide_session
-    def __get_task_concurrency_map(self, states, session=None):
+    def __get_concurrency_maps(self, states, session=None):
         """
-        Returns a map from tasks to number in the states list given.
+        Returns a map from dags to number in the states list given and
 
 Review comment:
   ```suggestion
           Returns a map from dags to number of task instances in the states list given and
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services