Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verkle-db backends #31

Open
kevaundray opened this issue Dec 4, 2021 · 2 comments
Open

Verkle-db backends #31

kevaundray opened this issue Dec 4, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@kevaundray
Copy link
Contributor

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
@kevaundray kevaundray added the enhancement New feature or request label Dec 12, 2021
@ControlCplusControlV
Copy link
Contributor

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
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants