-
Notifications
You must be signed in to change notification settings - Fork 683
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
feat(bench): add tooling for synthetic native transfer benchmarks #12570
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #12570 +/- ##
==========================================
- Coverage 70.48% 70.48% -0.01%
==========================================
Files 845 845
Lines 171985 171985
Branches 171985 171985
==========================================
- Hits 121227 121221 -6
- Misses 45644 45650 +6
Partials 5114 5114
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
I think it is fine to merge this in and we can make improvements as needed as the benchmarking evolves or when others who are using it find ways of making improvements to it.
Adds tooling to benchmark native token transfers with the
benchmark-native-transfer
command. The accounts involved in that benchmark can be generated withcreate-sub-accounts
. Examples of how to execute both commands are given in thejustfile
. Documentation is available inbenchmarking_synthetic_workloads.md
.The tooling is based on Rust and
near-jsonrpc-client
. Having the Rust toolchain installed should be sufficient to use it.This PR is rather big. Splitting it up into smaller pieces would result in parts that don't fully support running a workload. I assume that wouldn't be favorable.
The code added here is benchmark tooling only which doesn't affect
neard
, so I it shouldn't be able to break anything in production.