Skip to content

Commit

Permalink
Refactor: Remove sledstore as a storage backend
Browse files Browse the repository at this point in the history
The `sled` DB has known issues with memory and disk space management.
Even under relatively low load, it exhibits abnormal memory and disk
usage.

As a result, `sledstore` has been removed and is no longer recommended
as a storage backend.
  • Loading branch information
drmingdrmer committed Dec 10, 2024
1 parent 4aaff4f commit f0d240c
Show file tree
Hide file tree
Showing 7 changed files with 1 addition and 767 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,6 @@ jobs:
include:
- store: "stores/memstore"
- store: "stores/rocksstore"
- store: "stores/sledstore"

steps:
- name: Setup | Checkout
Expand Down
1 change: 0 additions & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ members = [
"tests",
"stores/memstore",
"stores/rocksstore",
"stores/sledstore",
]
exclude = [
"cluster_benchmark",
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ unused_dep:

typos:
# cargo install typos-cli
typos --write-changes openraft/ tests/ stores/memstore/ stores/rocksstore stores/sledstore examples/raft-kv-memstore/ examples/raft-kv-rocksdb/
typos --write-changes openraft/ tests/ stores/memstore/ stores/rocksstore examples/raft-kv-memstore/ examples/raft-kv-rocksdb/
#typos --write-changes --exclude change-log/ --exclude change-log.md --exclude derived-from-async-raft.md
# typos

Expand Down
33 changes: 0 additions & 33 deletions stores/sledstore/Cargo.toml

This file was deleted.

6 changes: 0 additions & 6 deletions stores/sledstore/README.md

This file was deleted.

Loading

0 comments on commit f0d240c

Please sign in to comment.