-
Notifications
You must be signed in to change notification settings - Fork 27
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
Jesse/loadtest legacy mode #92
Conversation
🙏 |
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.
It seems like we're only using this new logic in the case of a transaction and none of the other modes?
There are panics in testing and the and it seems like some performance issues.
577534e
to
2372100
Compare
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.
lgtm, great work @gatsbyz :)
I tested against geth
and didn't see any perf issues but let's wait for John's review :)
# standard
$ ./out/polycli loadtest --verbosity 700 --chain-id 1337 --concurrency 1 --requests 8000 --rate-limit 2000 --mode t http://127.0.0.1:8545
10:04AM INF * Results
10:04AM INF Samples samples=8000
10:04AM INF Start startTime=2023-06-29T10:04:42+02:00
10:04AM INF End endTime=2023-06-29T10:04:46+02:00
10:04AM INF Mean Wait meanWait=0.00042218980237499946
10:04AM INF Num errors numErrors=0
10:04AM INF Finished
# legacy
$ ./out/polycli loadtest --verbosity 700 --chain-id 1337 --concurrency 1 --requests 8000 --rate-limit 2000 --mode t --legacy http://127.0.0.1:8545
10:05AM INF * Results
10:05AM INF Samples samples=8000
10:05AM INF Start startTime=2023-06-29T10:05:02+02:00
10:05AM INF End endTime=2023-06-29T10:05:06+02:00
10:05AM INF Mean Wait meanWait=0.0004291837611249989
10:05AM INF Num errors numErrors=0
10:05AM INF Finished
* auto determine chain ID * chain id 0 * chain id command * Update cmd/loadtest/loadtest.go Co-authored-by: John Hilliard <[email protected]> * fix: minor duplicate log messages --------- Co-authored-by: John Hilliard <[email protected]>
* 1559 aware monitor variables * put back * add to struct
No description provided.