search

wuxibin89 / redis-go-cluster

redis cluster client implementation in Go
Apache License 2.0
488 stars 145 forks source link

Mget() works on slot level not on node level ? #33

Open mjmanav4 opened 5 years ago

mjmanav4 commented 5 years ago

The documentation says

Mutiple keys command - MGET/MSET are supported using result aggregation. Processing steps are as follows:

First, split the keys into multiple nodes according to their hash slot. Then, start a goroutine for each node to excute MGET/MSET commands and wait them finish. Last, collect and rerange all replies, return back to caller.

Then, start a goroutine for each node to excute MGET/MSET commands and wait them finish. is misleading number of goroutines is equal to no of hash-slots for the n keys, number of goroutines is not equal to number of nodes in cluster