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 2022/02/07 08:25:33 UTC

[GitHub] [airflow] ephraimbuddy edited a comment on pull request #21214: Removes the next_dagrun_create_after reset

ephraimbuddy edited a comment on pull request #21214:
URL: https://github.com/apache/airflow/pull/21214#issuecomment-1030925842


   Hi @avkirilishin, I debugged this issue and it turned out that the problem is not that the `next_dagrun_create_after` is being nullified. The problem is that this line https://github.com/apache/airflow/blob/1d170f899bcc87110e55192517270ec89d511ca8/airflow/jobs/scheduler_job.py#L1060 
   is returning a wrong figure. You can verify this. 
   
   I have not been able to pin out why it's returning a wrong figure but If you change this line of code:
   https://github.com/apache/airflow/blob/1d170f899bcc87110e55192517270ec89d511ca8/airflow/jobs/scheduler_job.py#L1088
   to
   ```python
   if self._should_update_dag_next_dagruns(dag, dag_model, active_runs-1):
   ```
   The problem will be gone.
   


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