writing manifest record regardless of wal option bool flag value. all…

… week 2 tests passing now
redixhumayun · Jun 7, 2024 · f2adac4 · f2adac4
1 parent 52f8fa6
commit f2adac4
Showing 1 changed file with 9 additions and 13 deletions.
diff --git a/mini-lsm-starter/src/lsm_storage.rs b/mini-lsm-starter/src/lsm_storage.rs
@@ -596,19 +596,15 @@ impl LsmStorageInner {
         state.imm_memtables.insert(0, current_memtable);
         state.memtable = new_memtable;
 
-        if self.options.enable_wal {
-            self.manifest
-                .as_ref()
-                .map(|manifest| {
-                    manifest.add_record(
-                        &_state_lock_observer,
-                        ManifestRecord::NewMemtable(memtable_id),
-                    )
-                })
-                .ok_or_else(|| {
-                    anyhow::anyhow!("no manifest handle found while freezing memtable")
-                })??;
-        }
+        self.manifest
+            .as_ref()
+            .map(|manifest| {
+                manifest.add_record(
+                    &_state_lock_observer,
+                    ManifestRecord::NewMemtable(memtable_id),
+                )
+            })
+            .ok_or_else(|| anyhow::anyhow!("no manifest handle found while freezing memtable"))??;
         self.sync_dir()?;
         Ok(())
     }