Adding a comment to no-op errors on failed vacuum

mergesort · Jan 22, 2024 · 7386ca7 · 7386ca7
1 parent 353d26e
commit 7386ca7
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/Sources/Bodega/SQLiteStorageEngine.swift b/Sources/Bodega/SQLiteStorageEngine.swift
@@ -211,7 +211,11 @@ public actor SQLiteStorageEngine: StorageEngine {
     /// Removes all the `Data` items from the database.
     public func removeAllData() throws {
         try self.connection.run(Self.storageTable.delete())
-        let _ = try? self.connection.vacuum()
+        do {
+            try self.connection.vacuum()
+        } catch {
+            // We don't want the function to throw if vacuuming fails, so we treat it like a no-op.
+        }
     }
 
     /// Checks whether a value with a key is persisted.