• debing.sun's avatar
    Have consistent behavior of SPUBLISH within multi/exec like regular command (#13276) · 9ffc35c9
    debing.sun authored
    
    
    This PR is based on the commits from PR #12944.
    
    Allow SPUBLISH command within multi/exec on replica
    
    Behavior on unstable:
    
    ```
    127.0.0.1:6380> CLUSTER NODES
    39ce8aa20f1f0d91f1a88d976ee1926dfefcdf1a 127.0.0.1:6380@16380 myself,slave 8b0feb120b68aac489d6a5af9c77dc40d71bc792 0 0 0 connected
    8b0feb120b68aac489d6a5af9c77dc40d71bc792 127.0.0.1:6379@16379 master - 0 1705091681202 0 connected 0-16383
    127.0.0.1:6380> SPUBLISH hello world
    (integer) 0
    127.0.0.1:6380> MULTI
    OK
    127.0.0.1:6380(TX)> SPUBLISH hello world
    QUEUED
    127.0.0.1:6380(TX)> EXEC
    (error) MOVED 866 127.0.0.1:6379
    ```
    
    With this change:
    
    ```
    127.0.0.1:6380> SPUBLISH hello world
    (integer) 0
    127.0.0.1:6380> MULTI
    OK
    127.0.0.1:6380(TX)> SPUBLISH hello world
    QUEUED
    127.0.0.1:6380(TX)> EXEC
    1) (integer) 0
    ```
    
    ---------
    Co-authored-by: default avatarHarkrishn Patro <harkrisp@amazon.com>
    Co-authored-by: default avataroranagra <oran@redislabs.com>
    9ffc35c9
cluster.c 55.6 KB