Changed the reply of BGSAVE and BGREWRITEAOF from +OK to a more meaningful message...

diff --git a/redis.c b/redis.c
index e6afdfcd140cffc415cee723b46f81d10ebeffde..98fbe22f13616f1ba3a08d0461fb16d41c57d9ec 100644 (file)
--- a/redis.c
+++ b/redis.c
@@ -3295,7 +3295,8 @@ static void bgsaveCommand(redisClient *c) {
         return;
     }
     if (rdbSaveBackground(server.dbfilename) == REDIS_OK) {
         return;
     }
     if (rdbSaveBackground(server.dbfilename) == REDIS_OK) {

-        addReply(c,shared.ok);
+        char *status = "+Background saving started\r\n";
+        addReplySds(c,sdsnew(status));

     } else {
         addReply(c,shared.err);
     }
     } else {
         addReply(c,shared.err);
     }


@@ -6082,7 +6083,8 @@ static void bgrewriteaofCommand(redisClient *c) {
         return;
     }
     if (rewriteAppendOnlyFileBackground() == REDIS_OK) {
         return;
     }
     if (rewriteAppendOnlyFileBackground() == REDIS_OK) {

-        addReply(c,shared.ok);
+        char *status = "+Background append only file rewriting started\r\n";
+        addReplySds(c,sdsnew(status));

     } else {
         addReply(c,shared.err);
     }
     } else {
         addReply(c,shared.err);
     }

diff --git a/redis.conf b/redis.conf
index 9bf1974b19f9ffba4a594df0eb4f178df5582516..90f1b7b60d38ae506acd451b24e8240faf14170b 100644 (file)
--- a/redis.conf
+++ b/redis.conf
@@ -130,6 +130,9 @@ databases 16
 # log file at startup ignoring the dump.rdb file.
 #
 # The name of the append only file is "appendonly.log"
 # log file at startup ignoring the dump.rdb file.
 #
 # The name of the append only file is "appendonly.log"

+#
+# IMPORTANT: Check the BGREWRITEAOF to check how to rewrite the append
+# log file in background when it gets too big.

 
 appendonly no
 
 
 appendonly no