feat: address comments

cluster.go

@@ -1053,10 +1053,7 @@ func (c *clusterClient) Dedicate() (DedicatedClient, func()) {
 func (c *clusterClient) Nodes() map[string]Client {
 	c.mu.RLock()
 	nodes := make(map[string]Client, len(c.conns))
-	disableCache := false
-	if (c.opt != nil && c.opt.DisableCache) || (c.rOpt != nil && c.rOpt.DisableCache) {
-		disableCache = true
-	}
+	disableCache := c.opt != nil && c.opt.DisableCache
 	for addr, cc := range c.conns {
 		nodes[addr] = newSingleClientWithConn(cc.conn, c.cmd, c.retry, disableCache)
 	}

helper_test.go

@@ -18,6 +18,17 @@ func TestMGetCache(t *testing.T) {
 		if err != nil {
 			t.Fatalf("unexpected err %v", err)
 		}
+		disableCacheClient, err := newSingleClient(&ClientOption{InitAddress: []string{""}, DisableCache: true}, m, func(dst string, opt *ClientOption) conn {
+			return m
+		})
+		if err != nil {
+			t.Fatalf("unexpected err %v", err)
+		}
+		t.Run("DisableCache", func(t *testing.T) {
+			if v, err := MGetCache(disableCacheClient, context.Background(), 100, []string{"1", "2"}); err != nil || v == nil {
+				t.Fatalf("unexpected response %v %v", v, err)
+			}
+		})
 		t.Run("Delegate DoCache", func(t *testing.T) {
 			m.DoMultiCacheFn = func(multi ...CacheableTTL) *redisresults {
 				if reflect.DeepEqual(multi[0].Cmd.Commands(), []string{"GET", "1"}) && multi[0].TTL == 100 &&

sentinel.go

@@ -177,10 +177,7 @@ func (c *sentinelClient) Dedicate() (DedicatedClient, func()) {
 
 func (c *sentinelClient) Nodes() map[string]Client {
 	conn := c.mConn.Load().(conn)
-	disableCache := false
-	if (c.mOpt != nil && c.mOpt.DisableCache) || (c.sOpt != nil && c.sOpt.DisableCache) {
-		disableCache = true
-	}
+	disableCache := c.mOpt != nil && c.mOpt.DisableCache
 	return map[string]Client{conn.Addr(): newSingleClientWithConn(conn, c.cmd, c.retry, disableCache)}
 }