redis-cluster-proxy icon indicating copy to clipboard operation
redis-cluster-proxy copied to clipboard

Redis Module supported?

Open lakano opened this issue 5 years ago • 3 comments

Hello, I love the idea of your proxy, thanks! Is the Redis modules are supported ? (eg: RediSearch)

lakano avatar May 03 '20 18:05 lakano

The proxy distribute clients' requests to redis node by calculating the slot of key in the requests. But the position of key in a request is uncertain for different module commands, so it doesn't know how to distribute clients' module requests. What's more, the proxy couldn't check arguments of module commands.

One idea, we add some module command items in config file. For one module command, it marks command name, first key, last key, key numbers, read/write and so on. Proxy can load these items, just register dynamic redis commands @artix75

ShooterIT avatar May 08 '20 12:05 ShooterIT

One idea, we add some module command items in config file. For one module command, it marks command name, first key, last key, key numbers, read/write and so on. Proxy can load these items, just register dynamic redis commands @artix75

Hi @lakano @artix75 I implemented it, I will submit the PR on this weekend. But for some modules, it may not support all commands that don't have key.

ShooterIT avatar May 15 '20 03:05 ShooterIT

Hi @lakano You could do some tests https://github.com/RedisLabs/redis-cluster-proxy/pull/65

ShooterIT avatar May 17 '20 10:05 ShooterIT