Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

Fix multiple Ping and assertion failure in Discovery #5483

Merged
merged 9 commits into from
Feb 19, 2019
Merged

Fix multiple Ping and assertion failure in Discovery #5483

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a939710
Skip sending Ping when we've already sent one to this node
gumb0 Feb 11, 2019
1dd33d6
Add unit test for addNode the same node twice
gumb0 Feb 11, 2019
6537567
Remove ignored replacement node in case trying to ping one evicted no…
gumb0 Feb 12, 2019
9dcc4da
Split ping() into synchronous ping() and asynchronous schedulePing()
gumb0 Feb 12, 2019
0daaaa5
Make m_allNodes contain only the nodes of the node table buckets.
gumb0 Feb 14, 2019
6b54ecf
Address minor review issues.
gumb0 Feb 14, 2019
bd42df4
Fix typos
gumb0 Feb 15, 2019
2cdb3f4
Fix tests after making m_allNodes contain only node table nodes
gumb0 Feb 15, 2019
ceffe80
Fix tests crash on clang build
gumb0 Feb 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading