-
Notifications
You must be signed in to change notification settings - Fork 198
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
Feat/trie sync fixes #2966
Feat/trie sync fixes #2966
Conversation
- added the possibility to switch from the 2 implementations from the config lines
…e-sync-optimization
- made the trie sync tests use real life storers
…e-sync-optimization
…e-sync-optimization
Fix trie sync OOM problems
Codecov Report
@@ Coverage Diff @@
## master #2966 +/- ##
==========================================
+ Coverage 74.89% 74.97% +0.07%
==========================================
Files 613 615 +2
Lines 58677 58857 +180
==========================================
+ Hits 43949 44130 +181
+ Misses 10797 10789 -8
- Partials 3931 3938 +7
Continue to review full report at Codecov.
|
- tweaked configs
# Conflicts: # data/syncer/userAccountsSyncer.go # data/trie/branchNode.go # data/trie/branchNode_test.go # data/trie/extensionNode.go # data/trie/extensionNode_test.go # data/trie/interface.go # data/trie/leafNode.go # data/trie/sync.go
More tries sync fixes
Trie sync optimization
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.
System test passed.
Sync trie fixes so the node will not raise an Out Of Memory exception during bootstrap
Added a new variant of trie syncer which should perform better