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

Integrate benchmark-block #5224

Merged
merged 9 commits into from
Apr 1, 2022
Merged

Integrate benchmark-block #5224

merged 9 commits into from
Apr 1, 2022

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Mar 30, 2022

Integrate the benchmark-block command to compare block execution time to consumed weight.
Example output from paritytech/substrate#11091:

Block 46 with  5299 tx used  67.04% of its weight ( 1,011,772,407 of  1,509,238,171 ns)    
Block 47 with  5299 tx used  65.75% of its weight (   992,298,518 of  1,509,238,171 ns)    
Block 48 with   208 tx used  69.95% of its weight (    47,328,881 of     67,660,429 ns)    
Block 49 with     2 tx used  69.93% of its weight (     6,524,096 of      9,329,057 ns)

Last one 😌. I will refactor afterwards to clean this up paritytech/substrate#11140
Currently working on a test.

Example invocation to measure some historic polkadot blocks:

./target/release/polkadot benchmark-block --from 9500000 --to 9500005 --execution wasm --wasm-execution compiled --pruning archive --db paritydb

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Mar 30, 2022
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez added B1-releasenotes 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. labels Mar 30, 2022
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
cli/src/command.rs Outdated Show resolved Hide resolved
cli/src/command.rs Show resolved Hide resolved
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Copy link
Member Author

ggwpez commented Mar 31, 2022

bot rebase

@paritytech-processbot
Copy link

Rebasing

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. 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.

3 participants