Make sure an automatic rewrite only gets triggered when no other child is in execution.

diff --git a/src/redis.c b/src/redis.c
index bf51db9d536df64fa675f22aea4565a63d19a217..33cef01500173d847e1ddc4938853b97dc512d40 100644 (file)
--- a/src/redis.c
+++ b/src/redis.c
@@ -692,14 +692,16 @@ int serverCron(struct aeEventLoop *eventLoop, long long id, void *clientData) {
          }
 
          /* Trigger an AOF rewrite if needed */
          }
 
          /* Trigger an AOF rewrite if needed */

-         if (server.auto_aofrewrite_perc &&
+         if (server.bgsavechildpid == -1 &&
+             server.bgrewritechildpid == -1 &&
+             server.auto_aofrewrite_perc &&

              server.appendonly_current_size > server.auto_aofrewrite_min_size)
          {
             int base = server.auto_aofrewrite_base_size ?
                             server.auto_aofrewrite_base_size : 1;
             long long growth = (server.appendonly_current_size*100/base);
             if (growth >= server.auto_aofrewrite_perc) {
              server.appendonly_current_size > server.auto_aofrewrite_min_size)
          {
             int base = server.auto_aofrewrite_base_size ?
                             server.auto_aofrewrite_base_size : 1;
             long long growth = (server.appendonly_current_size*100/base);
             if (growth >= server.auto_aofrewrite_perc) {

-                redisLog(REDIS_NOTICE,"Starting automatic rewriting of AOF on %lld growth",growth);
+                redisLog(REDIS_NOTICE,"Starting automatic rewriting of AOF on %lld%% growth",growth);

                 rewriteAppendOnlyFileBackground();
             }
          }
                 rewriteAppendOnlyFileBackground();
             }
          }