Merge pull request syndtr#351 from manish-sethi/fsync_new_manifest

Add fsync after creating new manifest

Author: syndtr

.travis.yml

@@ -1,12 +1,19 @@
 language: go
 
 go:
-  - 1.9.x
-  - 1.10.x
-  - 1.11.x
+  - 1.14.x
+  - 1.15.x
   - tip
 
 script:
-  - go vet ./...
+  - |
+    (set -e
+    if [ "$TRAVIS_GO_VERSION" != "tip" ]; then 
+      echo "executing go vet for go verion $TRAVIS_GO_VERSION"
+      go vet ./...
+    else
+      echo "skipping go vet for go verion $TRAVIS_GO_VERSION"
+    fi
+    )
   - go test -timeout 1h ./...
   - go test -timeout 30m -race -run "TestDB_(Concurrent|GoleveldbIssue74)" ./leveldb

leveldb/session_util.go

@@ -453,6 +453,12 @@ func (s *session) newManifest(rec *sessionRecord, v *version) (err error) {
 	if err != nil {
 		return
 	}
+	if !s.o.GetNoSync() {
+		err = writer.Sync()
+		if err != nil {
+			return
+		}
+	}
 	err = s.stor.SetMeta(fd)
 	return
 }