[QUESTION] Role change is not handled correctly about redis-plus-plus HOT 7 CLOSED

I did some research, and this is indeed a similar problem with the issue you mentioned. If you only call AsyncRedisCluster::redis to create an AsyncRedis object, and send command with this object, redis-plus-plus does not update the underlying node-slot mapping, and always send command to the old master.

Thanks for reporting the bug, and I'll try to fix it ASAP.

Before that, you can work around it by calling AsyncRedisCluster::get with the key you used, once you get an exception. This way, the underlying node-slot mapping will be updated, and redis-plus-plus will follow the MOVED indication.

Regards

from redis-plus-plus.

I fixed the problem, please try the latest code on dev branch. After some more tests, I'll merge the code into master branch.

You can use the following code to do a test:

auto cluster = RedisCluster("redis://127.0.0.1:7000");
while (true) {
    try {
        auto r = cluster.redis(key, false);
        cout << r.command<string>("ping") << endl;
    } catch (const Error &e) {
        cout << e.what() << endl;
    }

At the beginning, the test program prints "PONG". When the master that key located, is down, the program prints error message. When a new master is elected, the program prints "PONG" again.

You can try the latest fix. If you still has problem with it, feel free to let me know.

Sorry for the late fix. Too busy these days.

Regards

from redis-plus-plus.

Code has been merged into master branch. If you still have problem with it, feel free to reopen it.

Regards

from redis-plus-plus.

New release has been created. Enjoy it!

Regards

from redis-plus-plus.

Hi @sewenew , I'm facing the same issue, so please a quick correction would be more than welcome!

from redis-plus-plus.

Thanks a lot for the fix, could you please create a new release that includes the fix so as to incorporate it in our project?

from redis-plus-plus.

from redis-plus-plus.