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

Guard against invalid schedule loading #543

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

taf2
Copy link

@taf2 taf2 commented May 12, 2016

If a bad crontab is published to a schedule this will blow up and cause the process to become corrupted. This causes lots of schedules to be missed.

@meatballhat
Copy link
Member

@taf2 Hello! Can you rebase against master, and please edit your commit message to remove the profanity? Thank you! 😸

@meatballhat meatballhat self-assigned this May 25, 2016
@meatballhat meatballhat changed the title Safety Check Guard against invalid schedule loading Jun 26, 2016
taf2 and others added 4 commits March 19, 2017 12:56
better safe than sorry right?
log_error
… we have observed major throughput issues when jobs are dynamically scheduled adversly impacting throughput of delayed jobs
@iloveitaly
Copy link
Contributor

@taf2 friendly reminder on this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants