Verkle-db backends - Githubissues

crate-crypto / rust-verkle

Apache License 2.0

119 stars 41 forks source link

Verkle-db backends #31

Open kevaundray opened 2 years ago

kevaundray commented 2 years ago

This is a tracking issue for the following tasks:

impl SqLite as a key value and make the database optional
One can also impl Sqlite in verkle repo and use tables
Possibly add https://github.com/nomic-io/merk as a backend

ControlCplusControlV commented 2 years ago

Backends should also likely be moved to rust feature flags, so that downstream consumers can avoid compiling costly impls (such as rocksDB) if they only plan on using one

kevaundray commented 2 years ago

The verkle-db crate has the backends under a feature flag here: https://github.com/crate-crypto/rust-verkle/blob/master/verkle-db/Cargo.toml#L10

I should clarify that consumers do not need to implement or even use verkle-db since verkle-trie just needs somebody to implement the Db traits. I added verkle-db to make it easier for those that did not want to. In that light, we could make verkle-db optional, though not super important