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

Revert use of IAVL V1 & retract v0.47.10-kava.2 #547

Merged
merged 2 commits into from
May 28, 2024

Conversation

pirtleshell
Copy link
Member

The branch kava-v0.47.10-release-x should only ever use iavl v0.
Use of IAVL v1 should be performed on branch kava-release-v0.47.10-iavl-v1

Reverts the commits that switched to iavl v1. Added a retraction of v0.47.10-kava.2 to the go mod.

The branch `kava-v0.47.10-release-x` should only ever use iavl v0.
Use of IAVL v1 should be performed on branch `kava-release-v0.47.10-iavl-v1`.
the release incorrectly upgrades to iavl v1
hash := st.tree.Hash()
hash, err := st.tree.Hash()
if err != nil {
panic(err)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
Copy link
Member

@drklee3 drklee3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, so the net change from the commit before adding iavl v1 would be just the retract:

a5d0f96...rp-revert-iavl-v1

@pirtleshell pirtleshell merged commit 77e50b9 into kava-v0.47.10-release-x May 28, 2024
20 of 25 checks passed
@pirtleshell pirtleshell deleted the rp-revert-iavl-v1 branch May 28, 2024 23:04
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.

2 participants