fix(discovery): fix hanging if some peers don't respond
All threads resolved!
All threads resolved!
Previously the discovery algorithm would try to reach each known peer sequentially requiring each consequent request to succeed until the next one can be attempted. This would not work in some cases (see test in previous commit).
So the new algorithm instead makes a single attempt to reach each peer within a round, and if some failed they're retried in the next round of requests. This allows to overall discovery to succeed in cases when some of the initial peers never respond.
Closes #54 (closed)
Merge request reports
Activity
assigned to @gmoshkin
added 1 commit
- 0457f363 - fix(discovery): fix hanging if some peers don't respond
added 1 commit
- 51751f83 - fix(discovery): fix hanging if some peers don't respond
- Resolved by Georgy Moshkin
- Resolved by Georgy Moshkin
- Resolved by Yaroslav Dynnikov
- Resolved by Georgy Moshkin
- Resolved by Georgy Moshkin
added 1 commit
- fdb31625 - fix(discovery): fix hanging if some peers don't respond
added 1 commit
- 4d3116b0 - fix(discovery): fix hanging if some peers don't respond
requested review from @rosik
mentioned in issue #64 (closed)
Please register or sign in to reply