Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cluster): fix autopipeline with keyPrefix or arg array #1391

Merged
merged 1 commit into from
Aug 1, 2021

Commits on Jul 31, 2021

  1. fix(cluster): fix autopipeline with keyPrefix or arg array

    Previously, the building of pipelines ignored the key prefix.
    It was possible that two keys, foo and bar, might be set into
    the same pipeline. However, after being prefixed by a configured
    "keyPrefix" value, they may no longer belong to the same
    pipeline.
    
    This led to the error:
    "All keys in the pipeline should belong to the same slots
    allocation group"
    
    Similarly, `args[0]` can be a (possibly empty) array which the Command
    constructor would flatten. Properly compute the first flattened key when
    autopipelining for a Redis.Cluster instance.
    
    Amended version of https://github.com/luin/ioredis/pull/1335/files - see comments on that PR
    
    This may fix redis#1264 and redis#1248.
    
    Fixes redis#1392
    TysonAndre committed Jul 31, 2021
    Configuration menu
    Copy the full SHA
    f116d48 View commit details
    Browse the repository at this point in the history