search

RedisLabs / redis-cluster-proxy

A proxy for Redis clusters.
GNU Affero General Public License v3.0
990 stars 129 forks source link

Redis Module supported? #58

Open lakano opened 4 years ago

lakano commented 4 years ago

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

ShooterIT commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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