Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[sync] Add a limit to rollback block count, for avoid use too much memory #1773

Merged
merged 2 commits into from
Dec 8, 2020
Merged

[sync] Add a limit to rollback block count, for avoid use too much memory #1773

merged 2 commits into from
Dec 8, 2020

Conversation

jolestar
Copy link
Member

@jolestar jolestar commented Dec 7, 2020

Fix node panic for block_latency metric. resolve #1771

@jolestar
[sync] Add a limit to rollback block count, for avoid usage too memory.
c53b7bc 
Optimize BlockConnectedEvent, only try connect block when sync chain's total_difficulty > node's current chain.
@jolestar
[metric] Fix node panic for block_latency metric. resolve #1771
024f353
@jolestar jolestar changed the title [sync] Add a limit to rollback block count, for avoid usage too much memory [sync] Add a limit to rollback block count, for avoid use too much memory Dec 7, 2020
@jolestar jolestar requested a review from sanlee42 December 7, 2020 13:00
@ssyuan ssyuan merged commit 5d8345b into master Dec 8, 2020
@ssyuan ssyuan deleted the fix/sync_mem_usage branch December 8, 2020 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssyuan ssyuan ssyuan approved these changes

@tiangong3624749 tiangong3624749 Awaiting requested review from tiangong3624749

@sanlee42 sanlee42 Awaiting requested review from sanlee42 sanlee42 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

peer_message_handler panic
2 participants
@jolestar @ssyuan