uranusjr commented on code in PR #23662: URL: https://github.com/apache/airflow/pull/23662#discussion_r933023033
########## docs/apache-airflow/concepts/timetable.rst: ########## @@ -55,15 +55,17 @@ Built In Timetables Airflow comes with several common timetables built in to cover the most common use cases. Additional timetables may be available in plugins. +.. _CronTriggerTimetable + CronTriggerTimetable ^^^^^^^^^^^^^^^^^^^^^^^^^ -Another timetable which accepts a cron expression. It behaves more compliant with standard cron than ``CronDataIntervalTimetable``. +A cron compliant timetable. Review Comment: What does “cron compliant” mean? Cron is not a standard to be compliant to. Even if it is, this is still missing a lot of explaination what makes something cron compliant. I would probably suggest ditch this compliance idea altogether, and simply explain how this timetable actually behaves, and compare it to the cron daemon’s behaviour, and potentially differences. -- 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]
