From 8a0258aa4374212d8bf7004fa9022f821f6c93ed Mon Sep 17 00:00:00 2001 From: antirez Date: Fri, 13 Nov 2015 09:32:07 +0100 Subject: [PATCH] AOF: rewriting child killed by SIGUSR1 is not an error. --- src/aof.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/aof.c b/src/aof.c index 4d971ff10..9bfb9e906 100644 --- a/src/aof.c +++ b/src/aof.c @@ -1455,8 +1455,10 @@ void backgroundRewriteDoneHandler(int exitcode, int bysignal) { serverLog(LL_VERBOSE, "Background AOF rewrite signal handler took %lldus", ustime()-now); } else if (!bysignal && exitcode != 0) { - server.aof_lastbgrewrite_status = C_ERR; - + /* SIGUSR1 is whitelisted, so we have a way to kill a child without + * tirggering an error conditon. */ + if (bysignal != SIGUSR1) + server.aof_lastbgrewrite_status = C_ERR; serverLog(LL_WARNING, "Background AOF rewrite terminated with error"); } else {