Skip to content

Commit

Permalink
Rename mutex properties
Browse files Browse the repository at this point in the history
  • Loading branch information
dim committed Mar 30, 2015
1 parent f5091d4 commit e428ae1
Showing 1 changed file with 15 additions and 15 deletions.
30 changes: 15 additions & 15 deletions cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,12 @@ import (
type ClusterClient struct {
commandable

addrs map[string]struct{}
slots [][]string
sLock sync.RWMutex // protects slots & addrs cache
addrs map[string]struct{}
slots [][]string
slotsMx sync.RWMutex // protects slots & addrs cache

conns map[string]*Client
cLock sync.Mutex // protects conns
conns map[string]*Client
connMx sync.Mutex // protects conns

opt *ClusterOptions

Expand Down Expand Up @@ -47,8 +47,8 @@ func NewClusterClient(opt *ClusterOptions) (*ClusterClient, error) {

// Close closes the cluster connection
func (c *ClusterClient) Close() error {
c.sLock.Lock()
defer c.sLock.Unlock()
c.slotsMx.Lock()
defer c.slotsMx.Unlock()

return c.reset()
}
Expand All @@ -65,15 +65,15 @@ func (c *ClusterClient) getMasterAddrBySlot(hashSlot int) string {

// Returns a node's client for a given address
func (c *ClusterClient) getNodeClientByAddr(addr string) *Client {
c.cLock.Lock()
c.connMx.Lock()
client, ok := c.conns[addr]
if !ok {
opt := c.opt.clientOptions()
opt.Addr = addr
client = NewTCPClient(opt)
c.conns[addr] = client
}
c.cLock.Unlock()
c.connMx.Unlock()
return client
}

Expand All @@ -85,8 +85,8 @@ func (c *ClusterClient) process(cmd Cmder) {

hashSlot := hashSlot(cmd.clusterKey())

c.sLock.RLock()
defer c.sLock.RUnlock()
c.slotsMx.RLock()
defer c.slotsMx.RUnlock()

tried := make(map[string]struct{}, len(c.addrs))
addr := c.getMasterAddrBySlot(hashSlot)
Expand Down Expand Up @@ -150,8 +150,8 @@ func (c *ClusterClient) reloadIfDue() (err error) {

var infos []ClusterSlotInfo

c.sLock.Lock()
defer c.sLock.Unlock()
c.slotsMx.Lock()
defer c.slotsMx.Unlock()

// Try known addresses in random order (map interation order is random in Go)
// http://redis.io/topics/cluster-spec#clients-first-connection-and-handling-of-redirections
Expand All @@ -170,14 +170,14 @@ func (c *ClusterClient) reloadIfDue() (err error) {

// Closes all connections and flushes slots cache
func (c *ClusterClient) reset() (err error) {
c.cLock.Lock()
c.connMx.Lock()
for addr, client := range c.conns {
if e := client.Close(); e != nil {
err = e
}
delete(c.conns, addr)
}
c.cLock.Unlock()
c.connMx.Unlock()
c.slots = make([][]string, hashSlots)
return
}
Expand Down

0 comments on commit e428ae1

Please sign in to comment.