sidekiq-statistic icon indicating copy to clipboard operation
sidekiq-statistic copied to clipboard

Warning from Redis

Open danielgomezrico opened this issue 2 years ago • 3 comments

Im getting this warning everytime a job is proccessed on sidekiq:

Pipelining commands on a Redis instance is deprecated and will be removed in Redis 5.0.0.

redis.pipelined do
  redis.get("key")
end

should be replaced by

redis.pipelined do |pipeline|
  pipeline.get("key")
end

(called from /Users/xxxxxxxx/.rvm/gems/ruby-2.6.8@xxxxxxx/gems/sidekiq-statistic-1.4.0/lib/sidekiq/statistic/middleware.rb:40:in `block in save_entry_for_worker'}

danielgomezrico avatar May 24 '22 20:05 danielgomezrico

Same here - i guess the issue still exists

t1ll avatar Aug 24 '22 13:08 t1ll

+1

Environment:

  • sidekiq 6.5.6
  • sidekiq-statistic 1.4.0
  • redis 4.8.0

Error

Pipelining commands on a Redis instance is deprecated and will be removed in Redis 5.0.0.

redis.multi do
  redis.get("key")
end

should be replaced by

redis.multi do |pipeline|
  pipeline.get("key")
end

(called from /usr/local/bundle/gems/sidekiq-statistic-1.4.0/lib/sidekiq/statistic/middleware.rb:40:in `block in save_entry_for_worker'}"

reservedmuffin avatar Aug 31 '22 18:08 reservedmuffin

I can see that there are changes to fix that in the latest (2.0.0) release . But it's not available on the RubyGems. @davydovanton are there any plans to publish it there?

reservedmuffin avatar Aug 31 '22 18:08 reservedmuffin