thor
thor copied to clipboard
maindb v4
This PR upgrades maindb to v4. It greatly reduces the storage space occupied and significantly reduces the synchronization time.
- For MPT
- The encoding and storage format of nodes are comprehensively improved, reducing the storage space used to store the state trie by 30%. Moreover, the speed of encoding and decoding nodes has been greatly improved.
- Introducing the concept of Version to identify nodes, which simplifies the implementation code.
- Optimize the trie interface to make it easier to maintain.
- Remove leafbank stuff.
- Improve generation-based node cache to reduce the pressure of GC.
- Improve node storage key encoding.
- Optimize block indexing and storage format.
Check out commits for more detail.
Codecov Report
Attention: Patch coverage is 71.67530%
with 328 lines
in your changes are missing coverage. Please review.
Project coverage is 57.36%. Comparing base (
d1dcdea
) to head (0ebf3e7
). Report is 7 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #635 +/- ##
==========================================
+ Coverage 52.49% 57.36% +4.86%
==========================================
Files 156 154 -2
Lines 17218 16630 -588
==========================================
+ Hits 9038 9539 +501
+ Misses 7339 6181 -1158
- Partials 841 910 +69
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Please give more details of what testing is needed
- try sync with and without
-disable-pruner
, and record the time and disk space used. - test correctness of APIs
Hi @qianbin I was playing around with your changes and wanted to share some results about the sync performance.
Hi @qianbin I was playing around with your changes and wanted to share some results about the sync performance.
![]()
That's good. How about turning on -skip-logs
?
Hi @qianbin I was playing around with your changes and wanted to share some results about the sync performance.
That's good. How about turning on
-skip-logs
?
I'll re-run with this parameter and post the results
Hi @qianbin I was playing around with your changes and wanted to share some results about the sync performance.
That's good. How about turning on
-skip-logs
?
I'll re-run with skip logs and post the results again
This pull request has been marked as stale due to inactivity. It will be closed if no further activity occurs. Thank you for your contributions.