Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support time zones in CronExpression #68

Merged
merged 2 commits into from
Sep 20, 2019

Conversation

paulhilliar
Copy link
Contributor

Support time zones in CronExpression so that schedules can fire at a time of day that moves specific to the explicit zone not the zone of the JVM.

For example the expression "0 05 13,20 * * ?" (New York) should fire at 13:05/20:05 NY time every day, regardless of where the JVM sits.

…time of day that moves specific to the explicit zone not the zone of the JVM.
@kagkarlsson
Copy link
Owner

Excellent PR! Thanks for contributing!

Could you add a cron(String, ZoneId) method to the Schedules class as well?

…time of day that moves specific to the explicit zone not the zone of the JVM.

Incorporate review feedback - add Schedules.cron(String cronPattern, ZoneId zoneId)
@paulhilliar
Copy link
Contributor Author

Thanks. Added the Schedules method.

@kagkarlsson
Copy link
Owner

Thanks! Merging!

@kagkarlsson kagkarlsson merged commit 17b4d93 into kagkarlsson:master Sep 20, 2019
@kagkarlsson
Copy link
Owner

Should be available in maven central in version 6.1 soon

evenh pushed a commit to evenh/db-scheduler that referenced this pull request Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants