orpiske commented on code in PR #12098: URL: https://github.com/apache/camel/pull/12098#discussion_r1399495529
########## components/camel-quartz/src/main/docs/quartz-component.adoc: ########## @@ -164,6 +164,22 @@ quartz://groupName/timerName?cron=0+0/5+12-18+?+*+MON-FRI&trigger.timeZone=Europ The timeZone value is the values accepted by `java.util.TimeZone`. +== Specifying start date + +The Quartz Scheduler allows you to configure start date per trigger. You can provide the start date as TimeInMillis as follows: + +---- +quartz://groupName/timerName?cron=0+0/5+12-18+?+*+MON-FRI&trigger.startAt=1700452232554 +---- + +== Specifying end date + +The Quartz Scheduler allows you to configure end date per trigger. You can provide the end date as TimeInMillis as follows: + +---- +quartz://groupName/timerName?cron=0+0/5+12-18+?+*+MON-FRI&trigger.endAt=1700452232554 +---- + Review Comment: It's probably important do add note here stating that, when using a system clock such as `System.currentTimeMillis()`, users may be susceptible to problems caused by time drifts (such as when the system clock gets automatically adjusted after an inquiry to an NTP server) and unpredictable behavior during daylight saving time changes. Ideally, we should provide a duration that can be computed from a monotonic clock in complement to this. -- 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-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org