Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

remove MAGIC NUMBER timeout 1000 #764

Closed
wants to merge 1 commit into from

2 participants

@charsyam

Remove cluster timeout 1000 to define DEFAULT_REDIS_CLUSTER_TIMEOUT

@charsyam charsyam closed this
@charsyam charsyam deleted the charsyam:patch-2 branch
@JackieXie168 JackieXie168 referenced this pull request from a commit
@timmaxw timmaxw Don't leak memory when linux_tcp_conn_t constructor is interrupted. F…
…ixes some of the errors mentioned in issue #764.
b303587
@JackieXie168 JackieXie168 referenced this pull request from a commit
@timmaxw timmaxw In RPCDirectoryTest.Update: if connection was not established, abort …
…instead of evaluating an undefined expression. Closes #764.
fda9462
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 14, 2012
  1. @openbaas

    change MAGICNUMBER to define

    openbaas authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 1 deletion.
  1. +1 −1  src/cluster.c
  2. +1 −0  src/redis.h
View
2  src/cluster.c
@@ -1772,7 +1772,7 @@ void migrateCommand(redisClient *c) {
return;
if (getLongFromObjectOrReply(c,c->argv[4],&dbid,NULL) != REDIS_OK)
return;
- if (timeout <= 0) timeout = 1000;
+ if (timeout <= 0) timeout = DEFAULT_REDIS_CLUSTER_TIMEOUT;
/* Check if the key is here. If not we reply with success as there is
* nothing to migrate (for instance the key expired in the meantime), but
View
1  src/redis.h
@@ -91,6 +91,7 @@
#define REDIS_REPL_PING_SLAVE_PERIOD 10
#define REDIS_RUN_ID_SIZE 40
#define REDIS_OPS_SEC_SAMPLES 16
+#define DEFAULT_REDIS_CLUSTER_TIMEOUT 1000
/* Protocol and I/O related defines */
#define REDIS_MAX_QUERYBUF_LEN (1024*1024*1024) /* 1GB max query buffer. */
Something went wrong with that request. Please try again.