Do not try to flush on secondary instances

rocksdict · Nov 22, 2023 · f70a519 · f70a519
1 parent 5c5a5c9
commit f70a519
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/rdict.rs b/src/rdict.rs
@@ -1020,11 +1020,14 @@ impl Rdict {
     fn close(&mut self) -> PyResult<()> {
         let f_opt = &self.flush_opt;
         let db = self.get_db()?.borrow();
-        if let AccessTypeInner::ReadOnly { .. } = self.access_type.0 {
-            drop(db);
-            drop(self.column_family.take());
-            drop(self.db.take());
-            return Ok(());
+        match &self.access_type.0 {
+            AccessTypeInner::ReadOnly { .. } | AccessTypeInner::Secondary { .. } => {
+                drop(db);
+                drop(self.column_family.take());
+                drop(self.db.take());
+                return Ok(());
+            }
+            _ => (),
         };
         let flush_wal_result = db.flush_wal(true);
         let flush_result = if let Some(cf) = &self.column_family {