-
Notifications
You must be signed in to change notification settings - Fork 15
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
Dqlite LTS v1.17.1, go-dqlite v2 #202
Conversation
BenchmarkResults
Current status
|
26ee2d8
to
beefaa8
Compare
Review PerformanceSingle Nodehead(green): Dqlite LTS v1.17.1 Three NodesInterpretationThe single node test shows no significant discrepancy between the old and new Dqlite version for cpu, io reads & writes. The new dqlite version has a higher initial memory usage during the idle time, this is also true for the three node test. |
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.
Thanks a lot @louiseschmidtgen! LGTM!
Dqlite LTS, go-dqlite v2
Use the new Dqlite LTS v1.17.1 and go-dqlite v2.
This new version has a change that reduces the memory used for snapshots.
This PR waits on the Performance Test PR (#200) to confirm the performance as expected.