sidekiq-dynamic-queues icon indicating copy to clipboard operation
sidekiq-dynamic-queues copied to clipboard

Don't do a global alphabetical sort when strict ordering. Closes #4.

Open adstage-david opened this issue 10 years ago • 6 comments

I don't know if this is the intended functionality, but I need it for a worker setup like:

high_worker: sidekiq -q high.star.
worker: sidekiq -q medium.star. -q low.star.
metrics_worker: sidekiq -q high_metrics -q medium_metrics -q low_metrics
mailer_worker: sidekiq -q high_mailer -q medium_mailer -q low_mailer

With the current settings, lows will always end up before mediums due to the alphabetical sort after wildcards applied.

adstage-david avatar Mar 11 '14 00:03 adstage-david

I originally ported this from https://github.com/wr0ngway/resque-dynamic-queues and used https://github.com/wr0ngway/resque-queue-priority to dynamically handle priorities, so it made sense to sort the queues because that was the base resque behavior.

I didn't port resque-queue-priority to sidekiq as I stopped using sidekiq before it was needed. Perhaps it would be useful to you.

I've since merged those two into https://github.com/wr0ngway/qmore for use with Qless as that is what I'm working with now.

Given that I don't work with sidekiq anymore, are you interested in being added as a contributor and/or taking over ownership of this project? That way you can take it wherever you wish

wr0ngway avatar Mar 11 '14 13:03 wr0ngway

Sure thing - I think we'll be using this plugin at work for a while, don't have a problem helping contribute.

adstage-david avatar Mar 11 '14 16:03 adstage-david

Ok, added you as a collaborator. When doing a release, I typically, check in a new version.rb with a bumped version number (semantic versioning), then check in an updated changelog ( https://gist.github.com/wr0ngway/9504108 ), then run "rake release" Whats your rubygems id? I'll give you permissions there as well.

wr0ngway avatar Mar 12 '14 10:03 wr0ngway

I'm "therabidbanana" on rubygems - either that username or therabidbanana at gmail. When working on my home computer I'm also usually "therabidbanana" on github.

adstage-david avatar Mar 12 '14 17:03 adstage-david

@adstage-david I am also running into this problem now, any chance this can be included in a release?

djurre avatar Jul 14 '16 20:07 djurre

@djurre - like @wr0ngway - I've stopped using this gem... I can't recall the exact reason, but I believe it was related to our upgrading to Sidekiq Pro? Maybe reliable fetch is incompatible with this gem? Whatever the case, my suspicion that we'd keep using this gem was mistaken, so I haven't really had the chance to maintain it. :)

Given I don't have a real app to test this in any more/deal with the fallout of a bad release, I've been hesitant to roll a new version and push it up to rubygems. Maybe @wr0ngway can get you access if you're interested in keeping it up?

adstage-david avatar Aug 01 '16 16:08 adstage-david