Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork1.7k
(2.14) Repeating schedule on an interval#7504
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
neilalexander left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM but please add a test to show what happens when the message containing the schedule is deleted, aged out, purged etc.
Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
7cbbaa4 to465d5a4CompareMauriceVanVeen commentedNov 19, 2025
Extended the test ( Purging and aging out isn't explicitly tested I don't think, but probably doesn't need to be as the schedule itself needs to exist (be loaded from the store) in order for a message to be scheduled, if it doesn't exist (removed by any means) it's deactivated and removed from the hash wheel. |
Supports a repeating schedule on an interval as described inADR-51.
A schedule like
@every 1swill publish a message every second. A schedule can't run more quickly than once a second. For the case where the server has been down for a minute and it comes back up, it will not immediately trigger/spam 60 events. Instead it will only trigger once and continue on the same interval.Signed-off-by: Maurice van Veengithub@mauricevanveen.com