redis-cluster-client 0.16.6 → 0.16.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 201f719d07b33c0df8cde529564564e2993495a7817419730b6699047e40c068
-  data.tar.gz: 45790a9f41cd3708e8ffc7c2729e767204711c966cf9dd19aed419b7abc9fb5c
+  metadata.gz: e7b3e4681d84fa76e82207734150b10bfe7963c07f99988919cb6381bcc1be26
+  data.tar.gz: f00dfcd28b66eb101779396fc3237b385c03f1cd845bae82fd592ae34abe9746
 SHA512:
-  metadata.gz: '085310367927882a2e2d6547419ca4edc254ea2296411ed0c8f0f7352b48d98c54a6e005cc4e11ab0b5455810fa9b086dfefa9786373f926de520c7972bae5fe'
-  data.tar.gz: 5d3db4f8ecee72af9d44c239bab164c581b4983c21a7eca1063d3724f0f70d1ed5ef73d5b141a43d34a32edc6419210fd839c0d48b61efac18ac88d5306a5a03
+  metadata.gz: 4a0b9abb4d20edda5d576350d47f47fb0c181bc706d473ba39caa3faf0eae296e453ad25ee42debd4475cee32012260b1cc9753a9b382afb56392678dd4b9bbf
+  data.tar.gz: d272558440ca03e6473153cbfd7c39be0bcb0201c643801b4ff1950b0a6d2188d787e0a2c9e01efc596d82a088c2491e9a08e7efc9827ac9cf61c01152e8f878

data/lib/redis_client/cluster/command.rb

@@ -9,12 +9,11 @@ class RedisClient
     class Command
       EMPTY_STRING = ''
       EMPTY_HASH = {}.freeze
-      EMPTY_POLICIES = [nil, nil].freeze
       REQUEST_POLICY_PREFIX = 'request_policy:'
       RESPONSE_POLICY_PREFIX = 'response_policy:'
       SUBCOMMAND_DELIMITER = '|'
 
-      private_constant :EMPTY_HASH, :EMPTY_POLICIES, :REQUEST_POLICY_PREFIX,
+      private_constant :EMPTY_HASH, :REQUEST_POLICY_PREFIX,
                        :RESPONSE_POLICY_PREFIX, :SUBCOMMAND_DELIMITER
 
       # @see https://redis.io/docs/latest/commands/command/ The reply of the COMMAND command
@@ -105,15 +104,13 @@ class RedisClient
         end
 
         def build_spec(row)
-          request_policy, response_policy = parse_tips(row[7])
-
           ::RedisClient::Cluster::Command::Spec.new(
             first_key_position: parse_first_key_position(row),
             key_step: row[5],
             write?: parse_writability(row),
             readonly?: row[2].include?('readonly'),
-            request_policy: request_policy,
-            response_policy: response_policy,
+            request_policy: parse_policy_tip(row[7], REQUEST_POLICY_PREFIX),
+            response_policy: parse_policy_tip(row[7], RESPONSE_POLICY_PREFIX),
             subcommands: parse_subcommands(row[9])
           ).freeze
         end
@@ -137,20 +134,10 @@ class RedisClient
         end
 
         # The command tips are available in the redis 7.0 or later.
-        def parse_tips(tips)
-          return EMPTY_POLICIES if tips.nil? || tips.empty?
-
-          request_policy = response_policy = nil
-
-          tips.each do |tip|
-            if tip.start_with?(REQUEST_POLICY_PREFIX)
-              request_policy = -tip[REQUEST_POLICY_PREFIX.size..]
-            elsif tip.start_with?(RESPONSE_POLICY_PREFIX)
-              response_policy = -tip[RESPONSE_POLICY_PREFIX.size..]
-            end
-          end
-
-          [request_policy, response_policy]
+        # It returns a single value instead of a pair to avoid the allocation of the array.
+        def parse_policy_tip(tips, prefix)
+          tip = tips&.find { |t| t.start_with?(prefix) }
+          tip.nil? ? nil : -tip.delete_prefix(prefix)
         end
 
         # The information of the subcommands is available in the redis 7.0 or later.
@@ -164,7 +151,7 @@ class RedisClient
 
             # The server replies with a full name of the subcommand such as `xinfo|stream`.
             i = name.index(SUBCOMMAND_DELIMITER)
-            acc[i.nil? ? name : name[(i + 1)..]] = build_spec(row)
+            acc[i.nil? ? name : name[i + 1, name.size]] = build_spec(row)
           end.freeze
         end
       end

data/lib/redis_client/cluster/node_key.rb

@@ -26,7 +26,7 @@ class RedisClient
         pos = node_key.rindex(DELIMITER, -1)
         return [node_key, nil] if pos.nil?
 
-        [node_key[0, pos], node_key[(pos + 1)..]]
+        [node_key[0, pos], node_key[pos + 1, node_key.size]]
       end
 
       def split_bracketed(node_key)
@@ -36,7 +36,7 @@ class RedisClient
         return nil if end_bracket.nil?
 
         host = node_key[1, end_bracket - 1]
-        remainder = node_key[(end_bracket + 1)..]
+        remainder = node_key[end_bracket + 1, node_key.size]
         port = remainder.start_with?(DELIMITER) ? remainder[1..] : nil
         [host, port]
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redis-cluster-client
 version: !ruby/object:Gem::Version
-  version: 0.16.6
+  version: 0.16.7
 platform: ruby
 authors:
 - Taishi Kasuga