-
Notifications
You must be signed in to change notification settings - Fork 548
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
Add tool to test application of txs to ledger #14582
Add tool to test application of txs to ledger #14582
Conversation
60a8ceb
to
8da7d20
Compare
1811f12
to
8557d61
Compare
8557d61
to
f506b51
Compare
| Ok keypair -> | ||
keypair | ||
| Error err -> | ||
eprintf "Could not read the specified keypair: %s\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use the logger
above for all these eprintf
? what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I don't know where to get a serialized block myself. Should we have this run in CI (with small parameters)?
Add tool that creates a set number of 9-account-updates transactions and applies them to a given ledger.
f506b51
to
0045dd0
Compare
Block doesn't seem to impact the performance of the staged ledger application. Allowing to run without it simplifies the flow.
How to run the benchmark Assuming
where:
The output will look like:
Benchmark is structured in two phases:
So the benchmark simulates a single pass of ledger application of a full block. This is not a real benchmark, ina way that we run this only once. In future iterations I want to change the format and use the |
!ci-build-me |
Benchmark-like tool that creates a set number of 9-account-updates transactions and applies them to a given ledger.
Explain how you tested your changes:
How to use the tool:
Checklist: