-
Notifications
You must be signed in to change notification settings - Fork 290
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
Merge statedb flush #3323
Merge statedb flush #3323
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3323 +/- ##
==========================================
+ Coverage 32.18% 32.19% +0.02%
==========================================
Files 500 500
Lines 46785 46876 +91
Branches 21228 21321 +93
==========================================
+ Hits 15054 15089 +35
- Misses 17676 17787 +111
+ Partials 14055 14000 -55
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Benchmark for 20471beClick to view benchmark
|
Benchmark for 20471beClick to view benchmark
|
这几行方法应该是当初也想过保留多个 change set 遗留的,可以考虑做一下清理了。 |
Benchmark for 413d366Click to view benchmark
|
OK |
Benchmark for 460ef73Click to view benchmark
|
Benchmark for 006ce06Click to view benchmark
|
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.
Need more test case for change set.
4580b24
to
eae6d0d
Compare
Benchmark for 3185f9bClick to view benchmark
|
1. add compare continue commit and batch flush test 2. add remove test
Benchmark for 8ae0fe2Click to view benchmark
|
Benchmark for f78110aClick to view benchmark
|
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: N/A
What is the new behavior?
merge transactions of block flush, improve performance
Other information