jscheffl commented on code in PR #41321:
URL: https://github.com/apache/airflow/pull/41321#discussion_r1708088533
##########
airflow/models/dag.py:
##########
@@ -678,6 +678,12 @@ def __init__(
self.timetable = DatasetTriggeredTimetable(DatasetAll(*schedule))
self.schedule_interval = self.timetable.summary
elif isinstance(schedule, ArgNotSet):
+ warnings.warn(
Review Comment:
Just curious - maybe lagging history: Why was it never considered setting
`schedule=None` per default?
If I take a look to ALL your changes in all tests I see this seems the be
actually a common and reasonable default (whereas I acknowledge thatchanging
the default in a breaking change might confuse and irritate a LOT of users)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
