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

sc-transaction-pool: Always use best block to check if we should skip enactment #14285

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jun 1, 2023

We will calculate the tree route always against the best block and thus, we also should use this one to check if we should skip the checks.

… enactment

We will calculate the tree route always against the best block and thus, we also should use this one
to check if we should skip the checks.
@bkchr bkchr added A0-please_review Pull request needs code review. 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 B1-note_worthy Changes should be noted in the release notes T0-node This PR/Issue is related to the topic “node”. labels Jun 1, 2023
@bkchr bkchr requested a review from a team June 1, 2023 17:45
Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

👍

@bkchr bkchr requested a review from a team June 2, 2023 11:11
@bkchr bkchr merged commit 689da49 into master Jun 2, 2023
@bkchr bkchr deleted the bkchr-transaction-pool-enactment-stuff branch June 2, 2023 18:37
jasl added a commit to Phala-Network/khala-parachain that referenced this pull request Jun 3, 2023
jasl added a commit to Phala-Network/khala-parachain that referenced this pull request Jun 3, 2023
@jasl
Copy link
Contributor

jasl commented Jun 13, 2023

@bkchr could you help to backport this to polkadot-v0.9.33 branch?

nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
… enactment (paritytech#14285)

We will calculate the tree route always against the best block and thus, we also should use this one
to check if we should skip the checks.
crystalin pushed a commit to moonbeam-foundation/substrate that referenced this pull request Jul 19, 2023
… enactment (paritytech#14285)

We will calculate the tree route always against the best block and thus, we also should use this one
to check if we should skip the checks.
crystalin pushed a commit to moonbeam-foundation/substrate that referenced this pull request Jul 19, 2023
… enactment (paritytech#14285)

We will calculate the tree route always against the best block and thus, we also should use this one
to check if we should skip the checks.
Agusrodri pushed a commit to moonbeam-foundation/substrate that referenced this pull request Aug 15, 2023
… enactment (paritytech#14285)

We will calculate the tree route always against the best block and thus, we also should use this one
to check if we should skip the checks.
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. B1-note_worthy Changes should be noted in the 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 T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants