Skip to content

config: add rocksdb as a db backend option#4239

Merged
melekes merged 2 commits intomasterfrom
anton/3831
Dec 11, 2019
Merged

config: add rocksdb as a db backend option#4239
melekes merged 2 commits intomasterfrom
anton/3831

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Dec 11, 2019

Closes #3831

The support for rocksdb was added a while back in
tendermint/tm-db#12. This commit merely updates
the documentation.

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

Closes #3831

The support for rocksdb was added a while back in
tendermint/tm-db#12. This commit merely updates
the documentation.
@melekes melekes self-assigned this Dec 11, 2019
@melekes melekes added C:docs Component: Documentation R4R labels Dec 11, 2019
@tessr
Copy link
Contributor

tessr commented Dec 11, 2019

Thanks for catching this and updating the docs, @melekes!

One thing I wanted to check in on--I believe @marbar3778 had mentioned that the RocksDB integration wasn't fully completed (and we were even talking about removing it until it was more fully requested). Marko, can you catch me up on what you've been thinking there?

@tac0turtle
Copy link
Contributor

I believe it is already being used by some teams, mainly some tests are missing but no need to remove it. We recently switched from the fork to the main repo as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:docs Component: Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Rocksdb as a db option.

3 participants