redis-cluster-client 0.12.1 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5d90c095f5058e808331eaa9219fb52b9d4a27de639a9c54d194c68ecb253611
-  data.tar.gz: 8cd8092db87c7cfa42e00c6cabd30747bdea74130d8eb1e19b971b95637665fa
+  metadata.gz: 8aa9148e1dedc8ccb5087590b5b693ff2d32ab1b4a16bf25227ab0d030bea56c
+  data.tar.gz: f8d699a826f263cb5ff1d99f08fd1be1cec551f4306759e14165f800a7ffec57
 SHA512:
-  metadata.gz: 4821b78b0d5766566fa3943f65271908d3e52e3f82269bd2ea0a789e571df7818b5e0275c88d8833eb7b449c7c0eb3ed672034c1caf03fac35528114f31adb73
-  data.tar.gz: a4bb1d4a766cd742645c61342c2822e2bd337c792fb67b9793bfbb06ec0a889185eaa8b3ed34f96591f9e57997ecfe83cc901ce65451577b7e8f13d7d23f64b3
+  metadata.gz: 4630e1a9819484cf9675b4d6b77bcf39d437cd821ff3875c5717cfaccc827fc8482f71a6da41e5032227a02b19c214c339956d865ffa942eb8f901d329f3ffea
+  data.tar.gz: 3e2a2424421543b08623e76dcea06d2a646c3cfadbace64b6dce0adfd8bead2b24d78aa568d1ee36c45bc80c3cdcf473c7fa74e7f30976b0005f1d34791d6707

data/lib/redis_client/cluster/command.rb

@@ -3,7 +3,6 @@
 require 'redis_client'
 require 'redis_client/cluster/errors'
 require 'redis_client/cluster/key_slot_converter'
-require 'redis_client/cluster/normalized_cmd_name'
 
 class RedisClient
   class Cluster
@@ -30,7 +29,7 @@ class RedisClient
           nodes&.each do |node|
             regular_timeout = node.read_timeout
             node.read_timeout = slow_command_timeout > 0.0 ? slow_command_timeout : regular_timeout
-            reply = node.call('COMMAND')
+            reply = node.call('command')
             node.read_timeout = regular_timeout
             commands = parse_command_reply(reply)
             cmd = ::RedisClient::Cluster::Command.new(commands)
@@ -51,7 +50,7 @@ class RedisClient
           rows&.each_with_object({}) do |row, acc|
             next if row[0].nil?
 
-            acc[row[0].downcase] = ::RedisClient::Cluster::Command::Detail.new(
+            acc[row.first] = ::RedisClient::Cluster::Command::Detail.new(
               first_key_position: row[3],
               key_step: row[5],
               write?: row[2].include?('write'),
@@ -73,38 +72,59 @@ class RedisClient
       end
 
       def should_send_to_primary?(command)
-        name = ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_command(command)
-        @commands[name]&.write?
+        find_command_info(command.first)&.write?
       end
 
       def should_send_to_replica?(command)
-        name = ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_command(command)
-        @commands[name]&.readonly?
+        find_command_info(command.first)&.readonly?
       end
 
       def exists?(name)
-        @commands.key?(::RedisClient::Cluster::NormalizedCmdName.instance.get_by_name(name))
+        @commands.key?(name) || @commands.key?(name.to_s.downcase(:ascii))
       end
 
       private
 
-      def determine_first_key_position(command) # rubocop:disable Metrics/CyclomaticComplexity
-        case name = ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_command(command)
-        when 'eval', 'evalsha', 'zinterstore', 'zunionstore' then 3
-        when 'object' then 2
-        when 'memory'
+      def find_command_info(name)
+        @commands[name] || @commands[name.to_s.downcase(:ascii)]
+      end
+
+      def determine_first_key_position(command) # rubocop:disable Metrics/CyclomaticComplexity, Metrics/AbcSize, Metrics/PerceivedComplexity
+        if command.first.casecmp('get').zero?
+          find_command_info(command.first)&.first_key_position.to_i
+        elsif command.first.casecmp('mget').zero?
+          find_command_info(command.first)&.first_key_position.to_i
+        elsif command.first.casecmp('set').zero?
+          find_command_info(command.first)&.first_key_position.to_i
+        elsif command.first.casecmp('mset').zero?
+          find_command_info(command.first)&.first_key_position.to_i
+        elsif command.first.casecmp('del').zero?
+          find_command_info(command.first)&.first_key_position.to_i
+        elsif command.first.casecmp('eval').zero?
+          3
+        elsif command.first.casecmp('evalsha').zero?
+          3
+        elsif command.first.casecmp('zinterstore').zero?
+          3
+        elsif command.first.casecmp('zunionstore').zero?
+          3
+        elsif command.first.casecmp('object').zero?
+          2
+        elsif command.first.casecmp('memory').zero?
           command[1].to_s.casecmp('usage').zero? ? 2 : 0
-        when 'migrate'
+        elsif command.first.casecmp('migrate').zero?
           command[3].empty? ? determine_optional_key_position(command, 'keys') : 3
-        when 'xread', 'xreadgroup'
+        elsif command.first.casecmp('xread').zero?
+          determine_optional_key_position(command, 'streams')
+        elsif command.first.casecmp('xreadgroup').zero?
           determine_optional_key_position(command, 'streams')
         else
-          @commands[name]&.first_key_position.to_i
+          find_command_info(command.first)&.first_key_position.to_i
         end
       end
 
       def determine_optional_key_position(command, option_name)
-        idx = command.map { |e| e.to_s.downcase }.index(option_name&.downcase)
+        idx = command.map { |e| e.to_s.downcase(:ascii) }.index(option_name)
         idx.nil? ? 0 : idx + 1
       end
     end

data/lib/redis_client/cluster/concurrent_worker/on_demand.rb

@@ -5,6 +5,8 @@ class RedisClient
     module ConcurrentWorker
       class OnDemand
         def initialize(size:)
+          raise ArgumentError, "size must be positive: #{size}" unless size.positive?
+
           @q = SizedQueue.new(size)
         end
 

data/lib/redis_client/cluster/concurrent_worker/pooled.rb

@@ -11,6 +11,8 @@ class RedisClient
       # So it consumes memory 1 MB multiplied a number of workers.
       class Pooled
         def initialize(size:)
+          raise ArgumentError, "size must be positive: #{size}" unless size.positive?
+
           @size = size
           setup
         end

data/lib/redis_client/cluster/concurrent_worker.rb

@@ -71,14 +71,12 @@ class RedisClient
 
       module_function
 
-      def create(model: :on_demand, size: 5)
-        size = size.positive? ? size : 5
-
+      def create(model: :none, size: 5)
         case model
-        when :on_demand, nil then ::RedisClient::Cluster::ConcurrentWorker::OnDemand.new(size: size)
-        when :pooled then ::RedisClient::Cluster::ConcurrentWorker::Pooled.new(size: size)
         when :none then ::RedisClient::Cluster::ConcurrentWorker::None.new
-        else raise ArgumentError, "Unknown model: #{model}"
+        when :on_demand then ::RedisClient::Cluster::ConcurrentWorker::OnDemand.new(size: size)
+        when :pooled then ::RedisClient::Cluster::ConcurrentWorker::Pooled.new(size: size)
+        else raise ArgumentError, "unknown model: #{model}"
         end
       end
     end

data/lib/redis_client/cluster/errors.rb

@@ -43,11 +43,16 @@ class RedisClient
         if !errors.is_a?(Hash) || errors.empty?
           new(errors.to_s).with_errors(EMPTY_HASH)
         else
-          messages = errors.map { |node_key, error| "#{node_key}: (#{error.class}) #{error.message}" }
+          messages = errors.map { |node_key, error| "#{node_key}: (#{error.class}) #{error.message}" }.freeze
           new(messages.join(', ')).with_errors(errors)
         end
       end
 
+      def initialize(error_message = nil)
+        @errors = nil
+        super
+      end
+
       def with_errors(errors)
         @errors = errors if @errors.nil?
         self
@@ -61,7 +66,7 @@ class RedisClient
     end
 
     class NodeMightBeDown < Error
-      def initialize(_ = '')
+      def initialize(_error_message = nil)
         super(
           'The client is trying to fetch the latest cluster state ' \
           'because a subset of nodes might be down. ' \

data/lib/redis_client/cluster/node/latency_replica.rb

@@ -47,7 +47,7 @@ class RedisClient
               min = DUMMY_LATENCY_MSEC
               MEASURE_ATTEMPT_COUNT.times do
                 starting = obtain_current_time
-                cli.call_once('PING')
+                cli.call_once('ping')
                 duration = obtain_current_time - starting
                 min = duration if duration < min
               end

data/lib/redis_client/cluster/node.rb

@@ -90,7 +90,7 @@ class RedisClient
 
         def build_connection_prelude
           prelude = super.dup
-          prelude << ['READONLY'] if @scale_read
+          prelude << ['readonly'] if @scale_read
           prelude.freeze
         end
       end
@@ -309,7 +309,7 @@ class RedisClient
           work_group.push(i, raw_client) do |client|
             regular_timeout = client.read_timeout
             client.read_timeout = @config.slow_command_timeout > 0.0 ? @config.slow_command_timeout : regular_timeout
-            reply = client.call_once('CLUSTER', 'NODES')
+            reply = client.call_once('cluster', 'nodes')
             client.read_timeout = regular_timeout
             parse_cluster_node_reply(reply)
           rescue StandardError => e

data/lib/redis_client/cluster/optimistic_locking.rb

@@ -18,15 +18,15 @@ class RedisClient
         handle_redirection(slot, retry_count: 1) do |nd|
           nd.with do |c|
             c.ensure_connected_cluster_scoped(retryable: false) do
-              c.call('ASKING') if @asking
-              c.call('WATCH', *keys)
+              c.call('asking') if @asking
+              c.call('watch', *keys)
               begin
                 yield(c, slot, @asking)
               rescue ::RedisClient::ConnectionError
                 # No need to unwatch on a connection error.
                 raise
               rescue StandardError
-                c.call('UNWATCH')
+                c.call('unwatch')
                 raise
               end
             rescue ::RedisClient::CommandError => e

data/lib/redis_client/cluster/pipeline.rb

@@ -290,7 +290,7 @@ class RedisClient
       end
 
       def try_asking(node)
-        node.call('ASKING') == 'OK'
+        node.call('asking') == 'OK'
       rescue StandardError
         false
       end

data/lib/redis_client/cluster/pub_sub.rb

@@ -2,7 +2,6 @@
 
 require 'redis_client'
 require 'redis_client/cluster/errors'
-require 'redis_client/cluster/normalized_cmd_name'
 
 class RedisClient
   class Cluster
@@ -108,12 +107,21 @@ class RedisClient
 
       private
 
-      def _call(command)
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_command(command)
-        when 'subscribe', 'psubscribe', 'ssubscribe' then call_to_single_state(command)
-        when 'unsubscribe', 'punsubscribe' then call_to_all_states(command)
-        when 'sunsubscribe' then call_for_sharded_states(command)
-        else call_to_single_state(command)
+      def _call(command) # rubocop:disable Metrics/AbcSize
+        if command.first.casecmp('subscribe').zero?
+          call_to_single_state(command)
+        elsif command.first.casecmp('psubscribe').zero?
+          call_to_single_state(command)
+        elsif command.first.casecmp('ssubscribe').zero?
+          call_to_single_state(command)
+        elsif command.first.casecmp('unsubscribe').zero?
+          call_to_all_states(command)
+        elsif command.first.casecmp('punsubscribe').zero?
+          call_to_all_states(command)
+        elsif command.first.casecmp('sunsubscribe').zero?
+          call_for_sharded_states(command)
+        else
+          call_to_single_state(command)
         end
       end
 

data/lib/redis_client/cluster/router.rb

@@ -7,7 +7,6 @@ require 'redis_client/cluster/errors'
 require 'redis_client/cluster/key_slot_converter'
 require 'redis_client/cluster/node'
 require 'redis_client/cluster/node_key'
-require 'redis_client/cluster/normalized_cmd_name'
 require 'redis_client/cluster/transaction'
 require 'redis_client/cluster/optimistic_locking'
 require 'redis_client/cluster/pipeline'
@@ -37,33 +36,78 @@ class RedisClient
         raise
       end
 
-      def send_command(method, command, *args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-        cmd = ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_command(command)
-        case cmd
-        when 'ping'     then @node.send_ping(method, command, args).first.then(&TSF.call(block))
-        when 'wait'     then send_wait_command(method, command, args, &block)
-        when 'keys'     then @node.call_replicas(method, command, args).flatten.sort_by(&:to_s).then(&TSF.call(block))
-        when 'dbsize'   then @node.call_replicas(method, command, args).select { |e| e.is_a?(Integer) }.sum.then(&TSF.call(block))
-        when 'scan'     then scan(command, seed: 1)
-        when 'lastsave' then @node.call_all(method, command, args).sort_by(&:to_i).then(&TSF.call(block))
-        when 'role'     then @node.call_all(method, command, args, &block)
-        when 'config'   then send_config_command(method, command, args, &block)
-        when 'client'   then send_client_command(method, command, args, &block)
-        when 'cluster'  then send_cluster_command(method, command, args, &block)
-        when 'memory'   then send_memory_command(method, command, args, &block)
-        when 'script'   then send_script_command(method, command, args, &block)
-        when 'pubsub'   then send_pubsub_command(method, command, args, &block)
-        when 'watch'    then send_watch_command(command, &block)
-        when 'mset', 'mget', 'del'
-          send_multiple_keys_command(cmd, method, command, args, &block)
-        when 'acl', 'auth', 'bgrewriteaof', 'bgsave', 'quit', 'save'
+      def send_command(method, command, *args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity, Metrics/MethodLength
+        cmd_name = command.first
+        if cmd_name.casecmp('get').zero?
+          node = assign_node(command)
+          try_send(node, method, command, args, &block)
+        elsif cmd_name.casecmp('mget').zero?
+          send_multiple_keys_command(command.first, method, command, args, &block)
+        elsif cmd_name.casecmp('set').zero?
+          node = assign_node(command)
+          try_send(node, method, command, args, &block)
+        elsif cmd_name.casecmp('mset').zero?
+          send_multiple_keys_command(command.first, method, command, args, &block)
+        elsif cmd_name.casecmp('del').zero?
+          send_multiple_keys_command(command.first, method, command, args, &block)
+        elsif cmd_name.casecmp('ping').zero?
+          @node.send_ping(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('wait').zero?
+          send_wait_command(method, command, args, &block)
+        elsif cmd_name.casecmp('keys').zero?
+          @node.call_replicas(method, command, args).flatten.sort_by(&:to_s).then(&TSF.call(block))
+        elsif cmd_name.casecmp('dbsize').zero?
+          @node.call_replicas(method, command, args).select { |e| e.is_a?(Integer) }.sum.then(&TSF.call(block))
+        elsif cmd_name.casecmp('scan').zero?
+          scan(command, seed: 1)
+        elsif cmd_name.casecmp('lastsave').zero?
+          @node.call_all(method, command, args).sort_by(&:to_i).then(&TSF.call(block))
+        elsif cmd_name.casecmp('role').zero?
+          @node.call_all(method, command, args, &block)
+        elsif cmd_name.casecmp('config').zero?
+          send_config_command(method, command, args, &block)
+        elsif cmd_name.casecmp('client').zero?
+          send_client_command(method, command, args, &block)
+        elsif cmd_name.casecmp('cluster').zero?
+          send_cluster_command(method, command, args, &block)
+        elsif cmd_name.casecmp('memory').zero?
+          send_memory_command(method, command, args, &block)
+        elsif cmd_name.casecmp('script').zero?
+          send_script_command(method, command, args, &block)
+        elsif cmd_name.casecmp('pubsub').zero?
+          send_pubsub_command(method, command, args, &block)
+        elsif cmd_name.casecmp('watch').zero?
+          send_watch_command(command, &block)
+        elsif cmd_name.casecmp('acl').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('auth').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('bgrewriteaof').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('bgsave').zero?
           @node.call_all(method, command, args).first.then(&TSF.call(block))
-        when 'flushall', 'flushdb'
+        elsif cmd_name.casecmp('quit').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('save').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('flushall').zero?
+          @node.call_primaries(method, command, args).first.then(&TSF.call(block))
+        elsif cmd_name.casecmp('flushdb').zero?
           @node.call_primaries(method, command, args).first.then(&TSF.call(block))
-        when 'readonly', 'readwrite', 'shutdown'
-          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(cmd).with_config(@config)
-        when 'discard', 'exec', 'multi', 'unwatch'
-          raise ::RedisClient::Cluster::AmbiguousNodeError.from_command(cmd).with_config(@config)
+        elsif cmd_name.casecmp('readonly').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('readwrite').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('shutdown').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('discard').zero?
+          raise ::RedisClient::Cluster::AmbiguousNodeError.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('exec').zero?
+          raise ::RedisClient::Cluster::AmbiguousNodeError.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('multi').zero?
+          raise ::RedisClient::Cluster::AmbiguousNodeError.from_command(command.first).with_config(@config)
+        elsif cmd_name.casecmp('unwatch').zero?
+          raise ::RedisClient::Cluster::AmbiguousNodeError.from_command(command.first).with_config(@config)
         else
           node = assign_node(command)
           try_send(node, method, command, args, &block)
@@ -118,7 +162,7 @@ class RedisClient
         elsif e.message.start_with?('ASK')
           node = assign_asking_node(e.message)
           if retry_count >= 0
-            node.call('ASKING')
+            node.call('asking')
             retry
           end
         elsif e.message.start_with?('CLUSTERDOWN')
@@ -280,75 +324,106 @@ class RedisClient
         retry
       end
 
-      def send_config_command(method, command, args, &block)
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'resetstat', 'rewrite', 'set'
+      def send_config_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize
+        if command[1].casecmp('resetstat').zero?
           @node.call_all(method, command, args).first.then(&TSF.call(block))
-        else assign_node(command).public_send(method, *args, command, &block)
+        elsif command[1].casecmp('rewrite').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif command[1].casecmp('set').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
       def send_memory_command(method, command, args, &block)
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'stats' then @node.call_all(method, command, args, &block)
-        when 'purge' then @node.call_all(method, command, args).first.then(&TSF.call(block))
-        else assign_node(command).public_send(method, *args, command, &block)
+        if command[1].casecmp('stats').zero?
+          @node.call_all(method, command, args, &block)
+        elsif command[1].casecmp('purge').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
-      def send_client_command(method, command, args, &block)
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'list' then @node.call_all(method, command, args, &block).flatten
-        when 'pause', 'reply', 'setname'
+      def send_client_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize
+        if command[1].casecmp('list').zero?
+          @node.call_all(method, command, args, &block).flatten
+        elsif command[1].casecmp('pause').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif command[1].casecmp('reply').zero?
           @node.call_all(method, command, args).first.then(&TSF.call(block))
-        else assign_node(command).public_send(method, *args, command, &block)
+        elsif command[1].casecmp('setname').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
-      def send_cluster_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize
-        case subcommand = ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'addslots', 'delslots', 'failover', 'forget', 'meet', 'replicate',
-             'reset', 'set-config-epoch', 'setslot'
-          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', subcommand]).with_config(@config)
-        when 'saveconfig' then @node.call_all(method, command, args).first.then(&TSF.call(block))
-        when 'getkeysinslot'
+      def send_cluster_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+        if command[1].casecmp('addslots').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('delslots').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('failover').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('forget').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('meet').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('replicate').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('reset').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('set-config-epoch').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('setslot').zero?
+          raise ::RedisClient::Cluster::OrchestrationCommandNotSupported.from_command(['cluster', command[1]]).with_config(@config)
+        elsif command[1].casecmp('saveconfig').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif command[1].casecmp('getkeysinslot').zero?
           raise ArgumentError, command.join(' ') if command.size != 4
 
           handle_node_reload_error do
             node_key = @node.find_node_key_of_replica(command[2])
             @node.find_by(node_key).public_send(method, *args, command, &block)
           end
-        else assign_node(command).public_send(method, *args, command, &block)
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
-      def send_script_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'debug', 'kill'
+      def send_script_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+        if command[1].casecmp('debug').zero?
+          @node.call_all(method, command, args).first.then(&TSF.call(block))
+        elsif command[1].casecmp('kill').zero?
           @node.call_all(method, command, args).first.then(&TSF.call(block))
-        when 'flush', 'load'
+        elsif command[1].casecmp('flush').zero?
+          @node.call_primaries(method, command, args).first.then(&TSF.call(block))
+        elsif command[1].casecmp('load').zero?
           @node.call_primaries(method, command, args).first.then(&TSF.call(block))
-        when 'exists'
+        elsif command[1].casecmp('exists').zero?
           @node.call_all(method, command, args).transpose.map { |arr| arr.any?(&:zero?) ? 0 : 1 }.then(&TSF.call(block))
-        else assign_node(command).public_send(method, *args, command, &block)
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
       def send_pubsub_command(method, command, args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-        case ::RedisClient::Cluster::NormalizedCmdName.instance.get_by_subcommand(command)
-        when 'channels'
+        if command[1].casecmp('channels').zero?
           @node.call_all(method, command, args).flatten.uniq.sort_by(&:to_s).then(&TSF.call(block))
-        when 'shardchannels'
+        elsif command[1].casecmp('shardchannels').zero?
           @node.call_replicas(method, command, args).flatten.uniq.sort_by(&:to_s).then(&TSF.call(block))
-        when 'numpat'
+        elsif command[1].casecmp('numpat').zero?
           @node.call_all(method, command, args).select { |e| e.is_a?(Integer) }.sum.then(&TSF.call(block))
-        when 'numsub'
+        elsif command[1].casecmp('numsub').zero?
           @node.call_all(method, command, args).reject(&:empty?).map { |e| Hash[*e] }
                .reduce({}) { |a, e| a.merge(e) { |_, v1, v2| v1 + v2 } }.then(&TSF.call(block))
-        when 'shardnumsub'
+        elsif command[1].casecmp('shardnumsub').zero?
           @node.call_replicas(method, command, args).reject(&:empty?).map { |e| Hash[*e] }
                .reduce({}) { |a, e| a.merge(e) { |_, v1, v2| v1 + v2 } }.then(&TSF.call(block))
-        else assign_node(command).public_send(method, *args, command, &block)
+        else
+          assign_node(command).public_send(method, *args, command, &block)
         end
       end
 
@@ -369,17 +444,17 @@ class RedisClient
 
       def send_multiple_keys_command(cmd, method, command, args, &block) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
         # This implementation is prioritized performance rather than readability or so.
-        case cmd
-        when 'mget'
+        if cmd.casecmp('mget').zero?
           single_key_cmd = 'get'
           keys_step = 1
-        when 'mset'
+        elsif cmd.casecmp('mset').zero?
           single_key_cmd = 'set'
           keys_step = 2
-        when 'del'
+        elsif cmd.casecmp('del').zero?
           single_key_cmd = 'del'
           keys_step = 1
-        else raise NotImplementedError, cmd
+        else
+          raise NotImplementedError, cmd
         end
 
         return try_send(assign_node(command), method, command, args, &block) if command.size <= keys_step + 1 || ::RedisClient::Cluster::KeySlotConverter.hash_tag_included?(command[1])
@@ -402,10 +477,12 @@ class RedisClient
         end
 
         replies = pipeline.execute
-        result = case cmd
-                 when 'mset' then replies.first
-                 when 'del' then replies.sum
-                 else replies
+        result = if cmd.casecmp('mset').zero?
+                   replies.first
+                 elsif cmd.casecmp('del').zero?
+                   replies.sum
+                 else
+                   replies
                  end
         block_given? ? yield(result) : result
       end

data/lib/redis_client/cluster/transaction.rb

@@ -93,24 +93,24 @@ class RedisClient
       end
 
       def prepare_tx
-        @pipeline.call('MULTI')
+        @pipeline.call('multi')
         @pending_commands.each(&:call)
         @pending_commands.clear
       end
 
       def commit
-        @pipeline.call('EXEC')
+        @pipeline.call('exec')
         settle
       end
 
       def cancel
-        @pipeline.call('DISCARD')
+        @pipeline.call('discard')
         settle
       end
 
       def settle
         # If we needed ASKING on the watch, we need ASKING on the multi as well.
-        @node.call('ASKING') if @asking
+        @node.call('asking') if @asking
         # Don't handle redirections at this level if we're in a watch (the watcher handles redirections
         # at the whole-transaction level.)
         send_transaction(@node, redirect: !!@watching_slot ? 0 : MAX_REDIRECTION)
@@ -189,7 +189,7 @@ class RedisClient
       end
 
       def try_asking(node)
-        node.call('ASKING') == 'OK'
+        node.call('asking') == 'OK'
       rescue StandardError
         false
       end

data/lib/redis_client/cluster.rb

@@ -152,8 +152,9 @@ class RedisClient
     end
 
     def method_missing(name, *args, **kwargs, &block)
-      if router.command_exists?(name)
-        args.unshift(name)
+      cmd = name.respond_to?(:name) ? name.name : name.to_s
+      if router.command_exists?(cmd)
+        args.unshift(cmd)
         command = @command_builder.generate(args, kwargs)
         return router.send_command(:call_v, command, &block)
       end

data/lib/redis_client/cluster_config.rb

@@ -19,7 +19,7 @@ class RedisClient
     VALID_NODES_KEYS = %i[ssl username password host port db].freeze
     MERGE_CONFIG_KEYS = %i[ssl username password].freeze
     IGNORE_GENERIC_CONFIG_KEYS = %i[url host port path].freeze
-    MAX_WORKERS = Integer(ENV.fetch('REDIS_CLIENT_MAX_THREADS', 5))
+    MAX_WORKERS = Integer(ENV.fetch('REDIS_CLIENT_MAX_THREADS', -1)) # for backward compatibility
     # It's used with slow queries of fetching meta data like CLUSTER NODES, COMMAND and so on.
     SLOW_COMMAND_TIMEOUT = Float(ENV.fetch('REDIS_CLIENT_SLOW_COMMAND_TIMEOUT', -1))
     # It affects to strike a balance between load and stability in initialization or changed states.
@@ -110,12 +110,16 @@ class RedisClient
     private
 
     def merge_concurrency_option(option)
-      case option
-      when Hash
-        option = option.transform_keys(&:to_sym)
-        { size: MAX_WORKERS }.merge(option)
-      else { size: MAX_WORKERS }
+      opts = {}
+
+      if MAX_WORKERS.positive?
+        opts[:model] = :on_demand
+        opts[:size] = MAX_WORKERS
       end
+
+      opts.merge!(option.transform_keys(&:to_sym)) if option.is_a?(Hash)
+      opts[:model] = :none if opts.empty?
+      opts.freeze
     end
 
     def build_node_configs(addrs)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redis-cluster-client
 version: !ruby/object:Gem::Version
-  version: 0.12.1
+  version: 0.13.0
 platform: ruby
 authors:
 - Taishi Kasuga
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-11-18 00:00:00.000000000 Z
+date: 2024-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis-client
@@ -49,7 +49,6 @@ files:
 - lib/redis_client/cluster/node/random_replica_or_primary.rb
 - lib/redis_client/cluster/node_key.rb
 - lib/redis_client/cluster/noop_command_builder.rb
-- lib/redis_client/cluster/normalized_cmd_name.rb
 - lib/redis_client/cluster/optimistic_locking.rb
 - lib/redis_client/cluster/pipeline.rb
 - lib/redis_client/cluster/pub_sub.rb

data/lib/redis_client/cluster/normalized_cmd_name.rb

@@ -1,71 +0,0 @@
-# frozen_string_literal: true
-
-require 'singleton'
-
-class RedisClient
-  class Cluster
-    class NormalizedCmdName
-      include Singleton
-
-      EMPTY_STRING = ''
-
-      private_constant :EMPTY_STRING
-
-      def initialize
-        @cache = {}
-        @mutex = Mutex.new
-      end
-
-      def get_by_command(command)
-        get(command, index: 0)
-      end
-
-      def get_by_subcommand(command)
-        get(command, index: 1)
-      end
-
-      def get_by_name(name)
-        get(name, index: 0)
-      end
-
-      def clear
-        @mutex.synchronize { @cache.clear }
-        true
-      end
-
-      private
-
-      def get(command, index:)
-        name = extract_name(command, index: index)
-        return EMPTY_STRING if name.nil? || name.empty?
-
-        normalize(name)
-      end
-
-      def extract_name(command, index:)
-        case command
-        when String, Symbol then index.zero? ? command : nil
-        when Array then extract_name_from_array(command, index: index)
-        end
-      end
-
-      def extract_name_from_array(command, index:)
-        return if command.size - 1 < index
-
-        case e = command[index]
-        when String, Symbol then e
-        when Array then e[index]
-        end
-      end
-
-      def normalize(name)
-        return @cache[name] || name.to_s.downcase if @cache.key?(name)
-        return name.to_s.downcase if @mutex.locked?
-
-        str = name.to_s.downcase
-        @mutex.synchronize { @cache[name] = str }
-        str
-      end
-    end
-  end
-end