Clean up memories when syncing#1585
Conversation
|
@HoOngEe Would you shrink |
|
Before shrinking |
|
We have to shrink |
4147c49 to
134c094
Compare
|
@HoOngEe I know still there is a memory to cleaning up. However, I think it's hard to fix the problem, there is no apparent time to clean the cache. It should change the implementation to LRU like implementation. |
|
@sgkim126 Ok, let's merge this pr now and fix the cache implementation later. |
sgkim126
left a comment
There was a problem hiding this comment.
Please remove WIP tag when you think it's ready to merge.
core/src/verification/queue/mod.rs
Outdated
There was a problem hiding this comment.
Yes, It's redundant. I fixed it. Thank you.
29c49c8 to
603c211
Compare
Shrink HashMaps and Vectors of downloaders and Verification queue to reduce the memory usage when syncing.
603c211 to
0f4cded
Compare
|
@HoOngEe LGTM. Please apply it to rc-1.4.0 too. |


The commits in this pr piled upon the #1567
It closes #1576