From 56d85d44ad340e46a2e28d0c5e3745101ff0df77 Mon Sep 17 00:00:00 2001 From: Jason Wilder Date: Mon, 26 Oct 2015 13:16:32 -0600 Subject: [PATCH] Use RemoveAll instead of Remove When a database is dropped, removing old segments returns an error because the files are already gone. Using RemoveAll handles this case more gracefully. --- tsdb/engine/tsm1/wal.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tsdb/engine/tsm1/wal.go b/tsdb/engine/tsm1/wal.go index f4eb27fa60..558f721399 100644 --- a/tsdb/engine/tsm1/wal.go +++ b/tsdb/engine/tsm1/wal.go @@ -659,9 +659,9 @@ func (l *Log) flush(flush flushType) error { return err } if id <= lastFileID { - err := os.Remove(fn) + err := os.RemoveAll(fn) if err != nil { - return err + return fmt.Errorf("failed to remove: %v: %v", fn, err) } } }