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

ProofRecorder should support commited/uncommitted state #10222

Closed
bkchr opened this issue Nov 9, 2021 · 0 comments · Fixed by #13769
Closed

ProofRecorder should support commited/uncommitted state #10222

bkchr opened this issue Nov 9, 2021 · 0 comments · Fixed by #13769
Labels
J0-enhancement An additional feature request.

Comments

@bkchr
Copy link
Member

bkchr commented Nov 9, 2021

Currently the proof recorder will record any single access to the trie. The problem is if an extrinsic panics, we skip this extrinsic, but the proof recorder currently still keeps all the accessed trie nodes. In the case of a panic, we should also throw away the trie nodes that were recorded while trying to apply the failed tx.

@bkchr bkchr added the J0-enhancement An additional feature request. label Nov 9, 2021
@bkchr bkchr added this to SDK Node Apr 26, 2022
@bkchr bkchr moved this to Backlog 🗒 in SDK Node Apr 26, 2022
@RGafiyatullin RGafiyatullin self-assigned this Aug 18, 2022
@github-project-automation github-project-automation bot moved this from Backlog 🗒 to Done ✅ in SDK Node Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
J0-enhancement An additional feature request.
Projects
Status: done
Development

Successfully merging a pull request may close this issue.

2 participants