Fix CLUSTER REPLICATE error handling

microsoft / garnet

Garnet is a remote cache-store from Microsoft Research that offers strong performance (throughput and latency), scalability, storage, recovery, cluster sharding, key migration, and replication features. Garnet can work with existing Redis clients.

https://microsoft.github.io/garnet/

MIT License

9.71k stars 459 forks source link

Fix CLUSTER REPLICATE error handling #471

Closed kevin-montrose closed 1 week ago

kevin-montrose commented 1 week ago

CLUSTER REPLICATE would return +OK even if an error was encountered.

Adds a test for the failure, which replicates the conditions under which it was found (scripting spinning up a two node cluster).