-
Notifications
You must be signed in to change notification settings - Fork 597
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
Use tm-db master branch #1442
Use tm-db master branch #1442
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1442 +/- ##
=======================================
Coverage 19.60% 19.60%
=======================================
Files 242 242
Lines 32279 32279
=======================================
Hits 6329 6329
Misses 24792 24792
Partials 1158 1158 Continue to review full report at Codecov.
|
Thanks for this PR @faddat . This version of |
@p0mvn I don't know if there ever will be one. My understanding is that both the tendermint and cosmos teams would like to sunset it. |
I'm good with merging this, but if its alright I'd like us to get a smooth v9.0.0 release, and then try this out in a patch release? |
oh just read the changes in tm-db, lets just go ahead and merge this to main. I'm down to backport after an initial v9 release. Can go ahead and backport to v8 if someone wants to experiment with it |
Closes: #XXX
What is the purpose of the change
Working to overhaul tm-db upstream. This should enable the use of badgerdb v3, and soon (after another update) it will allow rocksdb to work
without manaully making a snapshot folder.
Brief change log
Testing and Verifying
(Please pick one of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Documentation and Release Note
Unreleased
section inCHANGELOG.md
? (yes / no) YESx/<module>/spec/
) / Osmosis docs repo / not documented)