-
Notifications
You must be signed in to change notification settings - Fork 1k
statement-store performance benchmarks #9763
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
Open
AndreiEres
wants to merge
43
commits into
master
Choose a base branch
from
AndreiEres/statement-store-bench
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
d7882cf
Add dummy test
AndreiEres cb8905e
Use only 2 validators
AndreiEres 521e145
Set many collators
AndreiEres 839256c
Send many statements
AndreiEres ba1102a
Save other session keys
AndreiEres 6bd0dd4
Exchange symmetric keys
AndreiEres aa77e39
Add dirty req-res
AndreiEres 016953c
Extract network spawning
AndreiEres 434bed2
Move requests to participant
AndreiEres 350bded
Request session keys only for group members
AndreiEres 88adee8
Add acks
AndreiEres 76df2bf
Add request acks
AndreiEres 12f5356
Remove request id from channels
AndreiEres 4fb7dcf
Add response acks
AndreiEres 13f5903
Refactor acks
AndreiEres 22bdaf4
Add retry to receiving
AndreiEres 0be0d9f
Remove assert
AndreiEres 83c5d65
Use async task for participants
AndreiEres 58d6ca4
Update logging
AndreiEres eb6863a
Test with more participants
AndreiEres ef85b4b
Switch to many participants
AndreiEres 677ba2c
Update
AndreiEres e575004
Increase statement-store limits
AndreiEres 4ec1174
Change log level
AndreiEres 1552999
Add stats
AndreiEres 45fd673
Use custom runtime
AndreiEres 9782eb9
Add basic stats
AndreiEres 19dfb34
Add timeouts
AndreiEres d18d673
Dry
AndreiEres 78ae251
Add RPC pool
AndreiEres 69c1e8a
Update
AndreiEres d6438c6
Update the bench
AndreiEres 26d5aee
Merge branch 'master' into AndreiEres/statement-store-bench
AndreiEres 3f4bd12
Merge remote-tracking branch 'origin/master' into AndreiEres/statemen…
AndreiEres 154a96f
Update the bench
AndreiEres d77bd22
Add memory stress test
AndreiEres 6815671
Update stress test
AndreiEres b4a17a4
Remove extra logic from the test
AndreiEres 6088289
Update network spawning
AndreiEres 208ba74
Update
AndreiEres c238d26
Fix clippy warnings
AndreiEres 5a8dcc8
Update formatting
AndreiEres db8380d
Update
AndreiEres File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
148 changes: 148 additions & 0 deletions
148
cumulus/zombienet/zombienet-sdk/tests/zombie_ci/people-rococo-spec.json
Large diffs are not rendered by default.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Why can't we use an existing spec how is this different ?
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.
if this is the binary George sent, I think we can just modify people-rococo to just have validate-statement implementation as needed for the benchmark, anyway rococo is not live AFAIK.
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.
Got it, so we need it because it is a rococo with statement-store pallet, let's just document the somewhere, so we don't forget what is in there.