Browse Source

Merge pull request #7945 from xiang90/snapshot_error

etcdserver: more logging on snapshot close path
Xiang Li 8 years ago
parent
commit
218ee40f11
1 changed files with 6 additions and 1 deletions
  1. 6 1
      etcdserver/snapshot_merge.go

+ 6 - 1
etcdserver/snapshot_merge.go

@@ -60,9 +60,14 @@ func newSnapshotReaderCloser(snapshot backend.Snapshot) io.ReadCloser {
 		n, err := snapshot.WriteTo(pw)
 		n, err := snapshot.WriteTo(pw)
 		if err == nil {
 		if err == nil {
 			plog.Infof("wrote database snapshot out [total bytes: %d]", n)
 			plog.Infof("wrote database snapshot out [total bytes: %d]", n)
+		} else {
+			plog.Warningf("failed to write database snapshot out [written bytes: %d]: %v", n, err)
 		}
 		}
 		pw.CloseWithError(err)
 		pw.CloseWithError(err)
-		snapshot.Close()
+		err = snapshot.Close()
+		if err != nil {
+			plog.Panicf("failed to close database snapshot: %v", err)
+		}
 	}()
 	}()
 	return pr
 	return pr
 }
 }