Skip to content

Conversation

@qcdll
Copy link
Contributor

@qcdll qcdll commented Dec 7, 2018

FYI purpose, not going to merge at the moment

@qcdll
Copy link
Contributor Author

qcdll commented Dec 7, 2018

in the end, tracemalloc's compare_to() in snapshot helped identify the root cause of memory leak in #252

we can use this PR in the future if memory issue arises again

@qcdll qcdll mentioned this pull request Dec 10, 2018
@qcdll qcdll force-pushed the mem-python3 branch 3 times, most recently from cbe583f to 3a0f825 Compare December 10, 2018 21:19
@qcdll
Copy link
Contributor Author

qcdll commented Dec 13, 2018

in addition to objgraph, tracking size of objects:

slave.add_block_futures
shard.state.new_block_pool
shard.add_block_futures
shard.state.db.m_header_pool
shard.state.db.m_meta_pool
shard.state.db.x_shard_set
shard.state.db.r_header_pool
shard.state.db.r_minor_header_pool
shard.state.db.height_to_minor_block_hashes

@qcdll
Copy link
Contributor Author

qcdll commented Dec 13, 2018

looks shard.state.db.height_to_minor_block_hashes has steady increase over time but the count is not out of ordinary, will revisit with new testnet 23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants