surrealdb / surrealkv

A low-level, versioned, embedded, ACID-compliant, key-value database for Rust
https://surrealdb.com
Apache License 2.0
293 stars 18 forks source link

Remove unused WAL #62

Closed gsserge closed 3 months ago

gsserge commented 3 months ago

Historically SurrealKV contained two log implementations, WAL and AOL. Only AOL is currently used, so it makes sense to remove WAL to simplify the codebase. If needed, the WAL implementation is still available in Git history.

@arriqaaq I've removed a few segments tests in storage/log/mod.rs because they were using WAL record header, and I'm not completely sure that that was the right call. Please let me know if I should keep them and use AOL segments instead.