diff --git a/src/cluster.c b/src/cluster.c index be89db32b19c7929db3179505f964daad6c67667..27c1f55a457b91afc3a6db7ee028cfee73fafaf0 100644 --- a/src/cluster.c +++ b/src/cluster.c @@ -1450,30 +1450,22 @@ int clusterDelSlot(int slot) { * Cluster state evaluation function * -------------------------------------------------------------------------- */ void clusterUpdateState(void) { - int ok = 1; int j; + /* Start assuming the state is OK. We'll turn it into FAIL if there + * are the right conditions. */ + server.cluster->state = REDIS_CLUSTER_OK; + /* Check if all the slots are covered. */ for (j = 0; j < REDIS_CLUSTER_SLOTS; j++) { if (server.cluster->slots[j] == NULL || server.cluster->slots[j]->flags & (REDIS_NODE_FAIL)) { - ok = 0; + server.cluster->state = REDIS_CLUSTER_FAIL; break; } } - /* Update cluster->state accordingly. */ - if (ok) { - if (server.cluster->state == REDIS_CLUSTER_NEEDHELP) { - server.cluster->state = REDIS_CLUSTER_NEEDHELP; - } else { - server.cluster->state = REDIS_CLUSTER_OK; - } - } else { - server.cluster->state = REDIS_CLUSTER_FAIL; - } - /* Compute the cluster size, that is the number of master nodes * serving at least a single slot. */ { diff --git a/src/redis.h b/src/redis.h index 276f07ead64015d4f3c774b2a33f1c02dd81ac72..27f8743da8c1d19342d7328ed3854e27b2ce4c8d 100644 --- a/src/redis.h +++ b/src/redis.h @@ -518,7 +518,6 @@ typedef struct redisOpArray { #define REDIS_CLUSTER_SLOTS 16384 #define REDIS_CLUSTER_OK 0 /* Everything looks ok */ #define REDIS_CLUSTER_FAIL 1 /* The cluster can't work */ -#define REDIS_CLUSTER_NEEDHELP 2 /* The cluster works, but needs some help */ #define REDIS_CLUSTER_NAMELEN 40 /* sha1 hex length */ #define REDIS_CLUSTER_PORT_INCR 10000 /* Cluster port = baseport + PORT_INCR */