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/08/31 07:39:07 UTC

[GitHub] [airflow] ephraimbuddy opened a new pull request, #26082: Fix schedule_interval in decorated dags

ephraimbuddy opened a new pull request, #26082:
URL: https://github.com/apache/airflow/pull/26082

   Using schedule_interval in decorated dag does not work at the moment. The
   issue is that the schedule arg on the dag decorator has a default of None thus when we
   have schedule_interval set and not schedule arg, the schedule arg resets the schedule_interval.
   
   The fix was to make the schedule arg same with what's in DAG __init__ as instructed


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


[GitHub] [airflow] uranusjr commented on pull request #26082: Fix schedule_interval in decorated dags

Posted by GitBox <gi...@apache.org>.
uranusjr commented on PR #26082:
URL: https://github.com/apache/airflow/pull/26082#issuecomment-1232584422

   ```
   # NOTE: When updating arguments here, please also keep arguments in @dag()
   # below in sync. (Search for 'def dag(' in this file.)
   ```
   
   Apparantly it’s still too easy to forget about this… Maybe a pre-commit check is needed. I’ll try to write one.


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


[GitHub] [airflow] ephraimbuddy merged pull request #26082: Fix schedule_interval in decorated dags

Posted by GitBox <gi...@apache.org>.
ephraimbuddy merged PR #26082:
URL: https://github.com/apache/airflow/pull/26082


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