redis 5 returns bool instead of int for sadd and srem https://github.…

…com/redis/redis-rb/blob/4.x/CHANGELOG.md#480
EverFi · May 6, 2024 · bc7ee8d · bc7ee8d
1 parent 2642ebd
commit bc7ee8d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/searchkick/relation_indexer.rb b/lib/searchkick/relation_indexer.rb
@@ -46,7 +46,7 @@ def batches_left
     end
 
     def batch_completed(batch_id)
-      Searchkick.with_redis { |r| r.srem(batches_key, batch_id) }
+      Searchkick.with_redis { |r| r.srem?(batches_key, batch_id) }
     end
 
     private
@@ -134,7 +134,7 @@ def full_reindex_async(relation)
     end
 
     def batch_job(class_name, batch_id, record_ids)
-      Searchkick.with_redis { |r| r.sadd(batches_key, batch_id) }
+      Searchkick.with_redis { |r| r.sadd?(batches_key, batch_id) }
       Searchkick::BulkReindexJob.perform_later(
         class_name: class_name,
         index_name: index.name,