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

Use storage transaction when running with proof recorder #10581

Merged
merged 1 commit into from
Jan 4, 2022

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jan 4, 2022

This will enable parachains to re-import a block without re-executing it.

Fixes: #4455

This will enable parachains to re-import a block without re-executing it.
@bkchr bkchr added 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. labels Jan 4, 2022
@bkchr bkchr added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label Jan 4, 2022
@arkpar
Copy link
Member

arkpar commented Jan 4, 2022

Wouldn't it prevent the proof recorder from recording the storage root calculation?

@bkchr
Copy link
Member Author

bkchr commented Jan 4, 2022

Wouldn't it prevent the proof recorder from recording the storage root calculation?

No. The transaction cache is just for collecting the "transaction" that is created as result of storage::root(), so we don't need to re-execute the block before importing it on the collator.

@bkchr bkchr merged commit 823581c into master Jan 4, 2022
@bkchr bkchr deleted the bkchr-remove-proving-backend branch January 4, 2022 19:48
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
…10581)

This will enable parachains to re-import a block without re-executing it.
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
…10581)

This will enable parachains to re-import a block without re-executing it.
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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transaction cache for proving backend
3 participants