sidekiq-debouncer 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 939e5397d600c7f5f24b3757f46f0c613369079ef327c8e88bf5adcdf807733f
-  data.tar.gz: 6ee4f636b9297a232a82bcf6c27c1418408cd2ad66ad8e473945fd1fdbc12cb0
+  metadata.gz: 78b258ab5ad21facbcf96f6aab96f30e38542ea24a7c5fd9e23ee99057b67e6d
+  data.tar.gz: 19db1ba7e392a4de9dcedfa8e236b8e8e334d7de56e71d8241aeb4e2d20b5e0a
 SHA512:
-  metadata.gz: 3b84978f7ea580d81a052719860ef38fc10339d35064c768457ffcccf0effb7dd148a2bd7a6620489a68c5a03705aea400f2ce8f493e2d0e27fa48edcb806082
-  data.tar.gz: f0e2d36120e11883f9b2f29b4de2f4fcbfbae075c72592742b9b6a3f323458a0bf7222fb4853604df45c1755e15ead8b600024630bd976631d3df4b5679f996a
+  metadata.gz: 4c30876a9db412ee65a844c966fec6b4f2c84e33f5eb0393773c7db0ad9fd5d6c394b4c9a2f01e31178c93dcc5d5443e829e03f90a9c1db81814f594153af290
+  data.tar.gz: 44e2d19a313ca3d07daaed130a9fb8a21721a6e6ed17756ed21028017096dd528d6b6a2d482a624095822c23158a9777620ae6dbeb0e541b8e2c7acebd003f68

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## [2.0.1] - 2023-03-04
+- don't remove debounce key in redis to avoid invalid debouncing
+
 ## [2.0.0] - 2023-02-28
 Complete rewrite of the library:  
 - Instead of iterating through whole schedule set, sidekiq-debouncer will now cache debounce key in redis with a reference to the job.

data/README.md

@@ -88,8 +88,8 @@ Keep in mind that the result of the debounce method will be converted to string,
 
 In the application, call `MyWorker.perform_async(...)` as usual. Everytime you call this function, `MyWorker`'s execution will be postponed by 5 minutes. After that time `MyWorker` will receive a method call `perform` with an array of arguments that were provided to the `MyWorker.perform_async(...)` calls.
 
-To avoid keeping leftover keys in redis (for example, when job was manually removed from schedule set), all additional keys are created with TTL.
-It's 7 days by default and should be ok in most of the cases. If you are debouncing your jobs in higher interval than that, you can overwrite this setting:
+To avoid keeping leftover keys in redis, all additional keys are created with TTL.
+It's 24 hours by default and should be ok in most of the cases. If you are debouncing your jobs in higher interval than that, you can overwrite this setting:
 
 ```ruby
 Sidekiq.configure_client do |config|

data/lib/sidekiq/debouncer/middleware/client.rb

@@ -17,7 +17,7 @@ module Sidekiq
         REDIS_ERROR_CLASS = defined?(RedisClient::CommandError) ? RedisClient::CommandError : Redis::CommandError
 
         def initialize(options = {})
-          @debounce_key_ttl = options.fetch(:ttl, 60 * 60 * 24 * 7) # 7 days by default
+          @debounce_key_ttl = options.fetch(:ttl, 60 * 60 * 24) # 24 hours by default
         end
 
         def call(worker_class, job, _queue, _redis_pool)

data/lib/sidekiq/debouncer/middleware/server.rb

@@ -3,20 +3,13 @@
 module Sidekiq
   module Debouncer
     module Middleware
-      # Server middleware removes debounce key from redis before executing the job
+      # wrap args into array because sidekiq uses splat while calling perform
       class Server
         include Sidekiq::ServerMiddleware
 
         def call(_worker, job, _queue)
           if job.key?("debounce_key")
-            # skip if job comes from dead or retry set
-            unless job.key?("failed_at")
-              redis do |connection|
-                connection.call("DEL", job["debounce_key"])
-              end
-            end
-
-            job["args"] = [job["args"]] # wrap args into array because sidekiq uses splat while calling perform
+            job["args"] = [job["args"]]
           end
 
           yield

data/lib/sidekiq/debouncer/version.rb

@@ -2,6 +2,6 @@
 
 module Sidekiq
   module Debouncer
-    VERSION = "2.0.0"
+    VERSION = "2.0.1"
   end
 end

data/sidekiq-debouncer.gemspec

@@ -12,6 +12,7 @@ Gem::Specification.new do |gem|
   gem.email = ["sebcioz@gmail.com", "kukicola@gmail.com"]
   gem.summary = "Sidekiq extension that adds the ability to debounce job execution"
   gem.description = <<~DESCRIPTION
+    Sidekiq extension that adds the ability to debounce job execution.
     Worker will postpone its execution after `wait time` have elapsed since the last time it was invoked.
     Useful for implementing behavior that should only happen after the input has stopped arriving.
   DESCRIPTION

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-debouncer
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Sebastian Zuchmański
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-28 00:00:00.000000000 Z
+date: 2023-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -102,6 +102,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.24.3
 description: |
+  Sidekiq extension that adds the ability to debounce job execution.
   Worker will postpone its execution after `wait time` have elapsed since the last time it was invoked.
   Useful for implementing behavior that should only happen after the input has stopped arriving.
 email: