ffinfo commented on a change in pull request #4396: [AIRFLOW-3585] - Add edges 
to database
URL: https://github.com/apache/airflow/pull/4396#discussion_r257452038
 
 

 ##########
 File path: airflow/www/views.py
 ##########
 @@ -92,7 +93,10 @@ def get_date_time_num_runs_dag_runs_form_data(request, 
session, dag):
     if dttm:
         dttm = pendulum.parse(dttm)
     else:
-        dttm = dag.latest_execution_date or timezone.utcnow()
+        execution_date = 
session.query(sqla.func.max(DagRun.execution_date)).filter(
+            DagRun.dag_id == dag.dag_id
+        ).scalar()
 
 Review comment:
   This should be temporary code. The problem here is that right now in 
views.py DAG and DagModel is used both for this method. `dag_id` is at least 
something both classes has in common. Accessing a method will break this. Once 
all is switched from DAG to DagModel this can be fixed again.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to