[GitHub] [airflow] jedcunningham commented on a change in pull request #19367: Do not send DAGs with null DagModel.max_active_runs to the scheduler

[GitBox <gi...@apache.org>]

jedcunningham commented on a change in pull request #19367:
URL: https://github.com/apache/airflow/pull/19367#discussion_r743970499



##########
File path: airflow/dag_processing/processor.py
##########
@@ -533,6 +533,10 @@ def update_import_errors(session: Session, dagbag: DagBag) -> None:
 
         # Add the errors of the processed files
         for filename, stacktrace in dagbag.import_errors.items():
+            dag_model = session.query(DagModel).filter(DagModel.fileloc == filename).first()

Review comment:
       I don't think this can be on just the first DagModel, don't we need to set it on all of them from that file?




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