search

enova / sidekiq-rate-limiter

Redis-backed, per-worker rate limits for job processing.
http://rubygems.org/gems/sidekiq-rate-limiter
MIT License
192 stars 43 forks source link

sidekiq 3.0.0 compatibility requested #3

Closed skull-squadron closed 10 years ago

skull-squadron commented 10 years ago 
17:57:14 web.1  | /Users/bmf/Projects/MyApp/vendor/bundle/ruby/2.1.0/gems/sidekiq-3.0.0/lib/sidekiq/fetch.rb:10:in `<class:Fetcher>': uninitialized constant Sidekiq::Fetcher::Util (NameError)
17:57:14 web.1  |   from /Users/bmf/Projects/MyApp/vendor/bundle/ruby/2.1.0/gems/sidekiq-3.0.0/lib/sidekiq/fetch.rb:9:in `<module:Sidekiq>'
17:57:14 web.1  |   from /Users/bmf/Projects/MyApp/vendor/bundle/ruby/2.1.0/gems/sidekiq-3.0.0/lib/sidekiq/fetch.rb:4:in `<top (required)>'
17:57:14 web.1  |   from /Users/bmf/Projects/MyApp/vendor/bundle/ruby/2.1.0/gems/sidekiq-rate-limiter-0.0.1/lib/sidekiq-rate-limiter/fetch.rb:2:in `<top (required)>'
17:57:14 web.1  |   from /Users/bmf/Projects/MyApp/vendor/bundle/ruby/2.1.0/gems/sidekiq-rate-limiter-0.0.1/lib/sidekiq-rate-limiter.rb:2:in `<top (required)>'
bwthomas commented 10 years ago

I will look into this as time allows. Hopefully it's as simple as adding a require.

mattes commented 10 years ago

:+1:

mattes commented 10 years ago 
require 'sidekiq/util'
require 'sidekiq-rate-limiter/server'

... fixes the uninitialized constant Sidekiq::Fetcher::Util (NameError) but throws undefined methodsidekiq_options'`.