Skip to content

Optimisations to squeeze more transactions into blocks #290

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

Closed
wants to merge 12 commits into from

Conversation

alexggh
Copy link
Contributor

@alexggh alexggh commented Mar 24, 2025

Remaining in this PR

  • Increase NORMAL_DISPATCH_RATIO to 100%, to not allways keep 75% empty, for testnetwork.
  • Reduce the Ref time for per read/write configuration to squeeze more createOrder in a block, this is needed because CreateOrder says it does 14 reads and 13 writes, so the configured reference time is what won't allow you to put more extrinsics in a block, with the default configuration reftime is about 2.7ms, so block won't include more 740 such transactions(2000ms/2.7), for testnetwork.

Split in other PRs:

Testing done:

  • Ran the network configuration in ./zombienet.sh testnet with a bloated state and myth-loader and observed authoring time on full block, it does not go over 1s.
  • Ran cumulus-pov-validator on all generated PoVs and all where within 1.5s, which is well within the validation backing deadline of 2.5s.

alexggh and others added 5 commits March 20, 2025 15:24
Signed-off-by: Alexandru Gheorghe <[email protected]>
Signed-off-by: Alexandru Gheorghe <[email protected]>
Signed-off-by: Alexandru Gheorghe <[email protected]>
@alexggh alexggh marked this pull request as ready for review April 4, 2025 06:53
@alexggh alexggh changed the title [WIP] Optimisations to squeze more transactions into blocks Optimisations to squeeze more transactions into blocks Apr 4, 2025
@Moliholy
Copy link
Collaborator

Moliholy commented Apr 9, 2025

Closing, as all commits in this PR have been included in #295.

@Moliholy Moliholy closed this Apr 9, 2025
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