-
-
Notifications
You must be signed in to change notification settings - Fork 208
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
Track processes in the database #421
Comments
Nice! This is a feature that resque has (although I don't think sidekiq actually does?) It is a somewhat error-prone and complex part of resque though, that ends up having a lot of lines of code behind it. Resque seems to have found it somewhat tricky to get right. |
@jrochkind thank you! I'll take a look at how Resque does it. Redis does have the capability to enumerate over all processes: https://www.rubydoc.info/github/mperham/sidekiq/Sidekiq/ProcessSet |
Considering this story done for the moment. Improvements are tracked in #560. To query processes: GoodJob::Process.all |
Awesome! Are these yet revealed in the admin dashboard? |
There should be a table of GoodJob processes in the database. They can be updated via a heartbeat by the Notifier, which should always hold a database connection. And a timeout value such that if the updated_at value hasn't been touched, the process would be treated as expired.
(Refactoring-wise: I think the notifier could become a more generalized 1-per-process supervisor that also contains the Poller's functionality too)
The text was updated successfully, but these errors were encountered: