Revert "leveldb: fix table file leaks when manifest is rotated (syndt…

…r#409)" This reverts commit 126854a.
zhang201702 · Jun 20, 2023 · 8ded095 · 8ded095
1 parent 63e8a13
commit 8ded095
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/go.mod b/go.mod
@@ -1,4 +1,4 @@
-module github.com/zhang201702/goleveldb
+module github.com/syndtr/goleveldb
 
 go 1.14
 

diff --git a/leveldb/session.go b/leveldb/session.go
@@ -223,12 +223,9 @@ func (s *session) commit(r *sessionRecord, trivial bool) (err error) {
 		}
 	}()
 
-	if s.manifest == nil {
+	if s.manifest == nil || s.manifest.Size() >= s.o.GetMaxManifestFileSize() {
 		// manifest journal writer not yet created, create one
 		err = s.newManifest(r, nv)
-	} else if s.manifest.Size() >= s.o.GetMaxManifestFileSize() {
-		// pass nil sessionRecord to avoid over-reference table file
-		err = s.newManifest(nil, nv)
 	} else {
 		err = s.flushManifest(r)
 	}