Use sadd? to avoid Redis warning

Avoids: Redis#sadd will always return an Integer in Redis 5.0.0. Use Redis#sadd? instead.
gzigzigzeo · Oct 25, 2022 · ea97b2d · ea97b2d
1 parent 1b2244d
commit ea97b2d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/sidekiq/grouping/redis.rb b/lib/sidekiq/grouping/redis.rb
@@ -14,9 +14,9 @@ class Redis
       def push_msg(name, msg, remember_unique = false)
         redis do |conn|
           conn.multi do |pipeline|
-            pipeline.sadd(ns("batches"), name)
+            pipeline.sadd?(ns("batches"), name)
             pipeline.rpush(ns(name), msg)
-            pipeline.sadd(unique_messages_key(name), msg) if remember_unique
+            pipeline.sadd?(unique_messages_key(name), msg) if remember_unique
           end
         end
       end