redis-migrate-tool icon indicating copy to clipboard operation
redis-migrate-tool copied to clipboard

rmt doesn't support distribution modula & random

Open patpatbear opened this issue 7 years ago • 2 comments

rmt redis_twem_backend_idx and redis_twem_backend_node use ketama distribution regardless of config. it will cause seriours data corruption if user use modula or random distribution.

redis_twem_backend_idx(redis_group *rgroup, uint8_t *key, uint32_t keylen)
{
    uint32_t idx, hash;
    int distribution = DIST_KETAMA;
    struct continuum *continuum;
    
   ...
}

patpatbear avatar Jun 08 '17 03:06 patpatbear

@patpatbear please check this out #22

iKenApple avatar Aug 10 '17 12:08 iKenApple

cool! seems fine for me ^_^ @iKenApple

patpatbear avatar Aug 11 '17 01:08 patpatbear