mik-laj commented on pull request #9544:
URL: https://github.com/apache/airflow/pull/9544#issuecomment-655809178


   I started looking at this change and I have three questions.
   1. What is your average difference in value between 
`dagrun.schedule_delay.<dag_id>` and 
`dagrun.<dag_id>.first_task_scheduling_delay`?
   1. From what I see, you fetch one task with one query. Have you tried to 
avoid it? It seems to me that you can have this data in your memory. Please 
look at: airflow/models/dagrun.py:295 (update_state method)
   1. Do you know why this change does not affect 
jobs.test_scheduler_job.TestDagFileProcessorQueriesCount? It seems to me that 
there should be a visible problem with too many queries, but for some reason 
this is not visible. 


----------------------------------------------------------------
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:
[email protected]


Reply via email to