re
/
redis
forked from mirror/redis
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #104 from go-redis/fix/use-cluster-info-to-ping 

cluster: user ClusterInfo instead of Ping to find live node.
This commit is contained in:
Vladimir Mihailenco 2015-05-14 15:06:30 +03:00
parent 368e3ce853 f24d3ff013
commit 6ee0a9b9f5
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cluster.go
View File

@ -97,7 +97,7 @@ func (c *ClusterClient) slotAddrs(slot int) []string {
return addrs
}
// randomClient returns a Client for the first pingable node.
// randomClient returns a Client for the first live node.
func (c *ClusterClient) randomClient() (client *Client, err error) {
for i := 0; i < 10; i++ {
n := rand.Intn(len(c.addrs))
@ -105,7 +105,7 @@ func (c *ClusterClient) randomClient() (client *Client, err error) {
if err != nil {
continue
}
err = client.Ping().Err()
err = client.ClusterInfo().Err()
if err == nil {
return client, nil
}

2
cluster_test.go
View File

@ -271,7 +271,7 @@ var _ = Describe("Cluster", func() {
Eventually(func() []string {
return client.SlotAddrs(slot)
}).Should(Equal([]string{"127.0.0.1:8221", "127.0.0.1:8224"}))
}, "5s").Should(Equal([]string{"127.0.0.1:8221", "127.0.0.1:8224"}))
})
It("should perform multi-pipelines", func() {