Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Fix state cache for cumulus#7990

Merged
bkchr merged 3 commits intomasterfrom
a-fix-cache
Jan 27, 2021
Merged

Fix state cache for cumulus#7990
bkchr merged 3 commits intomasterfrom
a-fix-cache

Conversation

@arkpar
Copy link
Member

@arkpar arkpar commented Jan 26, 2021

Fixes #7964

Cumulus commits same block to the database twice. State cache was not designed for this.

@arkpar arkpar requested a review from bkchr January 26, 2021 17:07
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jan 26, 2021
@arkpar arkpar added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jan 26, 2021
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
@bkchr bkchr merged commit 0b0d124 into master Jan 27, 2021
@bkchr bkchr deleted the a-fix-cache branch January 27, 2021 11:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected cache value for storage leading to state root mismatch

3 participants