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 2020/02/22 11:57:11 UTC

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #7489: [AIRFLOW-6869][WIP] Bulk fetch DAGRuns

nuclearpinguin commented on a change in pull request #7489: [AIRFLOW-6869][WIP] Bulk fetch DAGRuns
URL: https://github.com/apache/airflow/pull/7489#discussion_r382907514
 
 

 ##########
 File path: airflow/jobs/scheduler_job.py
 ##########
 @@ -700,22 +728,29 @@ def _process_dags(self, dagbag, dags, tis_out):
                 continue
 
             self.log.info("Processing %s", dag.dag_id)
+            dag_id = dag.dag_id
+            dag_runs_for_dag = list(dag_runs_by_dag_id[dag.dag_id]) if dag_id in dag_runs_by_dag_id else []
 
 Review comment:
   ```suggestion
               dag_runs_for_dag = list(dag_runs_by_dag_id[dag_id]) if dag_id in dag_runs_by_dag_id else []
   ```

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