-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Add cron
operator
#4192
Add cron
operator
#4192
Conversation
Most of the implementation is shared with the every operator
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.
Besides one minor functional issue, this works great. Thank you! 🚀
And sorry for the flood of comments. Most of them are nitpicks or about things that you'll just get used to very soon when it comes to this code base.
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.
Thank you for the changes. This works as expected now and can be merged once the remaining open conversations are addressed 👍
…into topic/scheduled-execution
Adds a
cron
operator that does scheduled execution analogous to theevery
operator.Fixes https://github.com/tenzir/issues/issues/1684