Deprecation warning: Pipelining commands on a Redis instance is deprecated and will be removed in Redis 5.0.0

[matthewhively]

Raised multiple times when performing Resque.enqueue by lib/redis/namespace.rb line 530 in version 1.8.2 of redis-namespace

I'm not sure whether this is an issue with the redis-namespace, or with the way that Resque itself is passing commands to the namespace.

[danlo]

Looks like this belongs to the resque gem: https://github.com/resque/resque/blob/master/lib/resque/data_store.rb#L103

[danlo]

The warning originates here:

https://github.com/redis/redis-rb/blob/master/lib/redis/pipeline.rb#L56

[danlo]

I looked though the redis-namespace code and found the following which may be of interest:

https://github.com/resque/redis-namespace/blob/master/lib/redis/namespace.rb#L529

    def namespaced_block(command, &block)
      if block.arity == 0
        redis.send(command, &block)
      else
        redis.send(command) do |r|
          copy = dup

Reading the redis documentation, I believe that an argument is always meant to always be passed.

https://github.com/redis/redis-rb#pipelining

Thus, block.arity == 0 should never happen?

-daniel

[ericboehs]

You're right, block.arity ==0 should never happen, but it is because whatever is calling your Redis connection is passing a block without an arg. The issue isn't with this gem (afaict), but with your app code.

Unfortunately the deprecation doesn't give a stack trace (even with ActiveSupport::Deprecation.debug = true set). In order to debug this, I edited the gem (bundle open redis-namespace) and threw a puts command, block in the arity == 0 if block (line 530).

This showed me the actual location of the bad code (e.g. calling .multi without an arg).

Removing the arity == 0 check here will just cause errors or push the deprecation to another line.

[mihael]

Am using redis with resque, and here's the problematic code:

https://github.com/resque/resque/blob/master/lib/resque/data_store.rb#L102-L106

[virgoproz]

Correct, data_store.rb in the current version of the Resque gem needs to be refactored to remove the deprecated use of .pipelined (without args)! Should be a no-brainer

[mishina2228]

Am using redis with resque, and here's the problematic code:

https://github.com/resque/resque/blob/master/lib/resque/data_store.rb#L102-L106

Now this part has been fixed in resque/resque#1806.
https://github.com/resque/resque/blob/ba5a44294ba5f6c4593deb78612f78989445d83e/lib/resque/data_store.rb#L102-L107

New version of resque has not yet been released though.

[kyrylo]

Can we push a new release, please? This warning is a little bit annoying since, in my case, it's being run almost after every single test that I have.

[nvolker]

Would be great to push this fix out. Appreciate the efforts here!

[kyrylo]

I can see v1.9.0 was pushed on August 13, 2022 but this warning has not been fixed for me.

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/kyrylosilin/.gem/ruby/2.7.6/gems/redis-namespace-1.9.0/lib/redis/namespace.rb:533:in `namespaced_block'}

[matthewhively]

Should be fixed by Resque 2.3.0
Not quite out yet.