fix

redis-rb · Nov 17, 2024 · 6e6c228 · 6e6c228
1 parent 1199747
commit 6e6c228
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/redis_client/cluster/command.rb b/lib/redis_client/cluster/command.rb
@@ -17,7 +17,6 @@ class Command
       Detail = Struct.new(
         'RedisCommand',
         :first_key_position,
-        :last_key_position,
         :key_step,
         :write?,
         :readonly?,
@@ -54,7 +53,6 @@ def parse_command_reply(rows)
 
             acc[row[0].downcase] = ::RedisClient::Cluster::Command::Detail.new(
               first_key_position: row[3],
-              last_key_position: row[4],
               key_step: row[5],
               write?: row[2].include?('write'),
               readonly?: row[2].include?('readonly')

diff --git a/test/redis_client/cluster/test_command.rb b/test/redis_client/cluster/test_command.rb
@@ -53,16 +53,16 @@ def test_parse_command_reply
               ['set', -3, Set['write', 'denyoom', 'movablekeys'], 1, -1, 2, Set['@write', '@string', '@slow'], Set[], Set[], Set[]]
             ],
             want: {
-              'get' => { first_key_position: 1, last_key_position: -1, key_step: 1, write?: false, readonly?: true },
-              'set' => { first_key_position: 1, last_key_position: -1, key_step: 2, write?: true, readonly?: false }
+              'get' => { first_key_position: 1, key_step: 1, write?: false, readonly?: true },
+              'set' => { first_key_position: 1, key_step: 2, write?: true, readonly?: false }
             }
           },
           {
             rows: [
               ['GET', 2, Set['readonly', 'fast'], 1, -1, 1, Set['@read', '@string', '@fast'], Set[], Set[], Set[]]
             ],
             want: {
-              'get' => { first_key_position: 1, last_key_position: -1, key_step: 1, write?: false, readonly?: true }
+              'get' => { first_key_position: 1, key_step: 1, write?: false, readonly?: true }
             }
           },
           { rows: [[]], want: {} },