Skip to content

Simplify shared rocksdb instance / write batch handling#3063

Merged
arnetheduck merged 1 commit intomasterfrom
fewer-pigs
Feb 14, 2025
Merged

Simplify shared rocksdb instance / write batch handling#3063
arnetheduck merged 1 commit intomasterfrom
fewer-pigs

Conversation

@arnetheduck
Copy link
Copy Markdown
Member

By introducing the "shared rocksdb instance" concept to the backend, we can remove the "piggybacking" mode , thus reducing the complexity of database initialisation and opening the possibility of extending how write batching works across kvt/aristo.

The change makes explicit the hidden shared state that was previously hiding in closures and provides the first step towards simplifying the "commit/persist" interface of coredb, preparing it for optimizations to reduce the "layering tax" that forked-layers introduced.

By introducing the "shared rocksdb instance" concept to the backend, we
can remove the "piggybacking" mode , thus reducing the complexity of
database initialisation and opening the possibility of extending how
write batching works across kvt/aristo.

The change makes explicit the hidden shared state that was previously
hiding in closures and provides the first step towards simplifying the
"commit/persist" interface of coredb, preparing it for optimizations to
reduce the "layering tax" that `forked-layers` introduced.
@arnetheduck arnetheduck merged commit 42bb640 into master Feb 14, 2025
@arnetheduck arnetheduck deleted the fewer-pigs branch February 14, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant