[
https://issues.apache.org/jira/browse/AIRFLOW-4269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16815808#comment-16815808
]
ASF subversion and git services commented on AIRFLOW-4269:
----------------------------------------------------------
Commit aeef5dfe2794c9e2b742625f4adf9b0ee4124fed in airflow's branch
refs/heads/master from Xiaodong
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=aeef5df ]
[AIRFLOW-4269] Minor acceleration of jobs._process_task_instances() (#5076)
* [AIRFLOW-4269] Minor acceleration of jobs._process_task_instances()
by breaking from unnecessary steps of a for-loop
* [AIRFLOW-4269] Improve log.info a bit
> For-lop in jobs._process_task_instances can be faster by breaking from
> unnecessary steps
> ----------------------------------------------------------------------------------------
>
> Key: AIRFLOW-4269
> URL: https://issues.apache.org/jira/browse/AIRFLOW-4269
> Project: Apache Airflow
> Issue Type: Improvement
> Affects Versions: 1.10.2
> Reporter: Xiaodong DENG
> Assignee: Xiaodong DENG
> Priority: Minor
>
> [https://github.com/apache/airflow/blob/64331aa5b90c44a3a22631d0ed168c44842a1fc8/airflow/jobs.py#L923]
> is a "continue" while it should be a break.
> When *len(active_dag_runs) >= dag.max_active_runs* , the following part of
> the for-loop will give the same result and result in a "continue" again and
> again, doing nothing.
> We should use a "break" here.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
