-
Notifications
You must be signed in to change notification settings - Fork 563
Improve Sidekiq queue efficiency #1381
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
Hi! I can help with this ✋ |
I think I can do this one. Have you done any work on this @v4vasanth? @BenEmdon So, your idea here is to just create these three queues that you've mentioned and re-assign the workers? I thought about doing something like this:
And assigning the Workers that have a queue with priority 3 to the Other option would be this setup:
But from the Sidekiq documentation we have that:
I don't like the idea of only processing from |
Feature request ✨
Currently classroom uses sidekiq queues inefficiently.
This is our current queue setup:
https://github.com/education/classroom/blob/4f9db7e412c57ce8aeb328a355b223e0a7254546/config/sidekiq.yml#L7-L14
This is what the Sidekiq docs say about having lots of queues:
One way we could improve this would be to use the regular 3 queues (
critical
,default
, andlow
).The text was updated successfully, but these errors were encountered: