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

Send parentchain extrinsics as mortal #3205

Merged
merged 9 commits into from
Dec 16, 2024

Conversation

Kailai-Wang
Copy link
Collaborator

Context

Try to mitigate the invalid tx situation by specifying them as mortal - the mortality at least gives the tx pool a chance to consider a tx as invalid once expired (and thus prune it from tx pool).

We should rarely need immortal extrinsic - so the current impl is definitely flawed.

In my local tests, it looks better - I don't see missing ParentchainBlockProcessed events anymore.

It doesn't solve all the problems tho, some tx will still be marked as invalid in the first submission with no reason being specified.

Ideally the validator_accessor and extrinsic_factory could be merged (as creating parentchain extrinsic requires parentchain state), but for now no big change is made until we rewrite it.

@Kailai-Wang Kailai-Wang self-assigned this Dec 13, 2024
Copy link

linear bot commented Dec 13, 2024

@Kailai-Wang Kailai-Wang requested a review from a team December 13, 2024 23:50
Copy link
Contributor

@silva-fj silva-fj left a comment

Choose a reason for hiding this comment

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

LGTM!

@Kailai-Wang Kailai-Wang merged commit 993f883 into dev Dec 16, 2024
22 checks passed
@Kailai-Wang Kailai-Wang deleted the p-1237-parentchain-extrinsics-piled-up branch December 16, 2024 10:01
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.

3 participants