-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Scheduler] add JitterTrigger
#50270
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
Conversation
c3e8464
to
79aa0a4
Compare
src/Symfony/Component/Scheduler/Trigger/CronExpressionTrigger.php
Outdated
Show resolved
Hide resolved
I'm wondering if we should do this at a higher level. Having a jitter is probably a common need for most triggers. |
Yes, I was thinking that also. |
79aa0a4
to
ae996e8
Compare
CronExpressionTrigger::jitter()
JitterTrigger
I've updated this PR to add a |
ebf05fb
to
6cec1d6
Compare
Co-authored-by: Uladzimir Tsykun <tsykun314@gmail.com>
6cec1d6
to
b0d984b
Compare
Thank you @kbond. |
I think this is what @vtsykun was trying to accomplish with #49806. FreeBSD has a
-jitter
option that does the same thing so I think there's precedent. From the (https://man.freebsd.org/cgi/man.cgi?cron):Usage: