From: Saj Goonatilleke <sg@redu.cx>
Date: Mon, 16 Jul 2012 05:33:25 +0000 (+1000)
Subject: Truncate short write from the AOF
X-Git-Url: https://git.saurik.com/redis.git/commitdiff_plain/4c0c1fff5a5abebfe903251ab56176b18cfc5688?ds=inline

Truncate short write from the AOF

If Redis only manages to write out a partial buffer, the AOF file won't
load back into Redis the next time it starts up.  It is better to
discard the short write than waste time running redis-check-aof.
---

diff --git a/src/aof.c b/src/aof.c
index 1df60b94..441ccaf1 100644
--- a/src/aof.c
+++ b/src/aof.c
@@ -250,6 +250,13 @@ void flushAppendOnlyFile(int force) {
                                    strerror(errno),
                                    (long)nwritten,
                                    (long)sdslen(server.aof_buf));
+
+            if (ftruncate(server.aof_fd, server.aof_current_size) == -1) {
+                redisLog(REDIS_WARNING, "Could not remove short write "
+                         "from the append-only file.  Redis may refuse "
+                         "to load the AOF the next time it starts.  "
+                         "ftruncate: %s", strerror(errno));
+            }
         }
         exit(1);
     }