-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
Scheduled workers #1
Comments
I actually never considered that but I will have a look and add it if needed. We really need the scheduled jobs to be unique as well so I'll get back to you on that. |
Any updates on this? |
Not yet, I've done some investigation and it looks like the recent sidekiq changes broke any unique possibility since every job is now unique. See sidekiq/sidekiq#359 for more information. |
It's currently not working very well if you schedule a job further than 3600 seconds into the future, I'll make the key expire when the unique job is scheduled for if it's a scheduled job. |
f89097e should fix this issue! |
Awesome! I plan to test this out either today or tomorrow |
Don't remove unique lock on shutdown
Was this intended to work with workers that need to be scheduled?
The text was updated successfully, but these errors were encountered: