jscheffl commented on PR #41397: URL: https://github.com/apache/airflow/pull/41397#issuecomment-2284672048
Thanks @uranusjr for the alternative proposal! This opens the door to "not deed" to make schedule argument as required as original PR and not needing to change default. I think this is a very good idea. I very much dislike (today) that there is a default schedule which is not matching 95% of (my) use cases. Actually a schedule of `None` would be very much reasonable for all defaults but of course coming from legacy this would be a breaking change. Main challenge that I see is: - This might need a bit of discussion and as 2.10 is almost closing to be completed I don't know if we can have a vote before release. I would be +1 for the change - Small feedback: Config setting is made with "negative phrases", maybe it would yield a bit better if working of config param is reversed/negated? -- 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]
