redis_queued_locks 1.21.0 → 1.22.0

data/Rakefile

@@ -55,4 +55,15 @@ end
 RSpec::Core::RakeTask.new(:rspec)
 Steep::RakeTask.new(:steep)
 
+# NOTE: (parallel specs) the runner: `spec/support/parallel_runner.rb`
+namespace :rspec do
+  desc 'Run the specs in parallel processes (WORKERS=<n>, each worker with its own Redis databases)'
+  task :parallel, [:paths] do |_task, args|
+    require_relative 'spec/support/parallel_runner'
+
+    workers = ENV.key?('WORKERS') ? Integer(ENV.fetch('WORKERS')) : nil
+    RQLParallelSpecs.run(workers, args[:paths].to_s.split)
+  end
+end
+
 task default: :rspec

data/lib/redis_queued_locks/acquirer/acquire_lock/instr_visitor.rb

@@ -2,7 +2,7 @@
 
 # @api private
 # @since 1.7.0
-# @version 1.21.0
+# @version 1.22.0
 # rubocop:disable Metrics/ModuleLength
 module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
   # rubocop:disable Metrics/ClassLength
@@ -11,6 +11,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer, NilClass]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -21,12 +22,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def extendable_reentrant_lock_obtained(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -36,7 +38,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.extendable_reentrant_lock_obtained', {
-        lock_key:, rw_mode:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
+        lock_key:, rw_mode:, priority:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
       }) rescue nil
     end
 
@@ -44,6 +46,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer, NilClass]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -54,12 +57,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def reentrant_lock_obtained(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -69,7 +73,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.reentrant_lock_obtained', {
-        lock_key:, rw_mode:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
+        lock_key:, rw_mode:, priority:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
       }) rescue nil
     end
 
@@ -77,6 +81,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer, NilClass]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -87,12 +92,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def lock_obtained(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -102,7 +108,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.lock_obtained', {
-        lock_key:, rw_mode:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
+        lock_key:, rw_mode:, priority:, ttl:, acq_id:, hst_id:, ts:, acq_time:, instrument:
       }) rescue nil
     end
 
@@ -110,6 +116,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer, NilClass]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -121,12 +128,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def reentrant_lock_hold_completes(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -137,7 +145,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.reentrant_lock_hold_completes', {
-        hold_time:, ttl:, acq_id:, hst_id:, ts:, lock_key:, rw_mode:, acq_time:, instrument:
+        hold_time:, ttl:, acq_id:, hst_id:, ts:, lock_key:, rw_mode:, priority:, acq_time:,
+        instrument:
       }) rescue nil
     end
 
@@ -145,6 +154,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer, NilClass]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -156,12 +166,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def lock_hold_and_release(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -172,7 +183,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.lock_hold_and_release', {
-        hold_time:, ttl:, acq_id:, hst_id:, ts:, lock_key:, rw_mode:, acq_time:, instrument:
+        hold_time:, ttl:, acq_id:, hst_id:, ts:, lock_key:, rw_mode:, priority:, acq_time:,
+        instrument:
       }) rescue nil
     end
 
@@ -183,6 +195,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     # @param instr_sampled [Boolean]
     # @param lock_key [String]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param ttl [Integer]
     # @param acq_id [String]
     # @param hst_id [String]
@@ -194,11 +207,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     #
     # @api private
     # @since 1.21.0
+    # @version 1.22.0
     def lock_obtaining_failed(
       instrumenter,
       instr_sampled,
       lock_key,
       rw_mode,
+      priority,
       ttl,
       acq_id,
       hst_id,
@@ -209,7 +224,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::InstrVisitor
     )
       return unless instr_sampled
       instrumenter.notify('redis_queued_locks.lock_obtaining_failed', {
-        lock_key:, rw_mode:, ttl:, acq_id:, hst_id:, ts:, acq_time:, result:, instrument:
+        lock_key:, rw_mode:, priority:, ttl:, acq_id:, hst_id:, ts:, acq_time:, result:,
+        instrument:
       }) rescue nil
     end
   end

data/lib/redis_queued_locks/acquirer/acquire_lock/log_visitor.rb

@@ -2,7 +2,7 @@
 
 # @api private
 # @since 1.7.0
-# @version 1.21.0
+# @version 1.22.0
 # rubocop:disable Metrics/ModuleLength
 module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
   # rubocop:disable Metrics/ClassLength
@@ -15,11 +15,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param host_id [String]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def start_lock_obtaining(
       logger,
       log_sampled,
@@ -28,7 +29,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       acquirer_id,
       host_id,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -39,7 +41,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "acq_id => '#{acquirer_id}' " \
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
-        "rw_mode => '#{rw_mode}'"
+        "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority}"
       end rescue nil
     end
 
@@ -51,11 +54,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param host_id [String]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def start_try_to_lock_cycle(
       logger,
       log_sampled,
@@ -64,7 +68,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       acquirer_id,
       host_id,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -75,7 +80,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "acq_id => '#{acquirer_id}' " \
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
-        "rw_mode => '#{rw_mode}'"
+        "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority}"
       end rescue nil
     end
 
@@ -87,11 +93,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param host_id [String]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def dead_score_reached__reset_acquirer_position(
       logger,
       log_sampled,
@@ -100,7 +107,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       acquirer_id,
       host_id,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -111,7 +119,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "acq_id => '#{acquirer_id}' " \
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
-        "rw_mode => '#{rw_mode}'"
+        "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority}"
       end rescue nil
     end
 
@@ -124,11 +133,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param acq_time [Numeric]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def extendable_reentrant_lock_obtained(
       logger,
       log_sampled,
@@ -138,7 +148,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       host_id,
       acq_time,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -150,6 +161,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
         "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority} " \
         "acq_time => #{acq_time} (ms)"
       end rescue nil
     end
@@ -163,11 +175,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param acq_time [Numeric]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def reentrant_lock_obtained(
       logger,
       log_sampled,
@@ -177,7 +190,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       host_id,
       acq_time,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -189,6 +203,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
         "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority} " \
         "acq_time => #{acq_time} (ms)"
       end rescue nil
     end
@@ -202,11 +217,12 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param acq_time [Numeric]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @return [void]
     #
     # @api private
     # @since 1.7.0
-    # @version 1.18.0
+    # @version 1.22.0
     def lock_obtained(
       logger,
       log_sampled,
@@ -216,7 +232,8 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       host_id,
       acq_time,
       access_strategy,
-      rw_mode
+      rw_mode,
+      priority
     )
       return unless log_sampled
 
@@ -228,6 +245,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
         "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority} " \
         "acq_time => #{acq_time} (ms)"
       end rescue nil
     end
@@ -244,11 +262,13 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
     # @param acq_time [Numeric]
     # @param access_strategy [Symbol]
     # @param rw_mode [Symbol]
+    # @param priority [Integer] The lock request priority.
     # @param result [Symbol] The failure result (`:timeout_reached`, `:retry_limit_reached`, ...).
     # @return [void]
     #
     # @api private
     # @since 1.21.0
+    # @version 1.22.0
     def lock_obtaining_failed(
       logger,
       log_sampled,
@@ -259,6 +279,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
       acq_time,
       access_strategy,
       rw_mode,
+      priority,
       result
     )
       return unless log_sampled
@@ -271,6 +292,7 @@ module RedisQueuedLocks::Acquirer::AcquireLock::LogVisitor
         "hst_id => '#{host_id}' " \
         "acs_strat => '#{access_strategy}' " \
         "rw_mode => '#{rw_mode}' " \
+        "priority => #{priority} " \
         "acq_time => #{acq_time} (ms) " \
         "result => '#{result}'"
       end rescue nil