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

rmt doesn't support distribution modula & random #18

Open
patpatbear opened this issue Jun 8, 2017 · 2 comments
Open

rmt doesn't support distribution modula & random #18

patpatbear opened this issue Jun 8, 2017 · 2 comments

Comments

@patpatbear
Copy link

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;
    
   ...
}
@iKenApple
Copy link

iKenApple commented Aug 10, 2017

@patpatbear please check this out #22

@patpatbear
Copy link
Author

cool! seems fine for me ^_^ @iKenApple

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants