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

Fix-up store #206

Merged
merged 3 commits into from
Aug 3, 2017
Merged

Fix-up store #206

merged 3 commits into from
Aug 3, 2017

Conversation

ethanfrey
Copy link
Contributor

I realized in the store overhaul, that KVCache no longer syncs in a deterministic order (as I had hoped not to need it for the merkle Tree). That means that in multiple node setups, one would soon have consensus failure as the AppHash would diverge even for the same contents. (But not in the one-node unit tests...)

Added a test case to ensure that applying changes to DeliverTx and then Committing them produces a deterministic hash, and now order the key sync properly.

@ethanfrey ethanfrey requested a review from cloudhead August 3, 2017 15:41
@ethanfrey ethanfrey merged commit 5d45c22 into unstable Aug 3, 2017
@ethanfrey ethanfrey deleted the feature/fixup-store branch August 21, 2017 21:06
yihuang added a commit to yihuang/cosmos-sdk that referenced this pull request Mar 27, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
dudong2 referenced this pull request in b-harvest/cosmos-sdk Oct 17, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
dudong2 referenced this pull request in b-harvest/cosmos-sdk Oct 17, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 12, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 12, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 16, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 16, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 16, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers

Problem: store key type assertion is incorrect (cosmos#244)

fix and add test
mmsqe pushed a commit to mmsqe/cosmos-sdk that referenced this pull request Dec 16, 2024
generic interface

generic btree

generic cachekv

generic transient store

support ObjStore

changelog

Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

object store key

Apply review suggestions

fix merge conflict

fix snapshot

revert dependers

Problem: store key type assertion is incorrect (cosmos#244)

fix and add test
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