-
Notifications
You must be signed in to change notification settings - Fork 524
test: large generated accountdb cluster test #4772
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
Conversation
algorandskiy
left a comment
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.
Could you also post command line on how to generate the db and run a test with it?
| } | ||
|
|
||
| //initial state | ||
| log := logging.NewLogger() |
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.
this inits local var, not global one. Maybe move it into init() ?
|
@brianolson please rebase on master to fix the build, and add usage examples into the PR description (what to run cli to have 30M accounts pregenerated before the test) |
Codecov Report
@@ Coverage Diff @@
## master #4772 +/- ##
==========================================
- Coverage 54.68% 54.68% -0.01%
==========================================
Files 414 414
Lines 53550 53561 +11
==========================================
+ Hits 29286 29291 +5
- Misses 21836 21842 +6
Partials 2428 2428
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Summary
Repair features for generating pre-built account database with many accounts already in it (to test sqlite3 account db performance at different sizes).
Test Plan
This is used in cluster tests.