Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

Peer eviction strategy #230

Open
Tracked by #3905
achingbrain opened this issue Oct 1, 2021 · 3 comments · Fixed by #229
Open
Tracked by #3905

Peer eviction strategy #230

achingbrain opened this issue Oct 1, 2021 · 3 comments · Fixed by #229

Comments

@achingbrain
Copy link
Member

JS does not favor long lived peers.

  1. We need to implement ping. When a useful peer becomes known, we add it to the routing table if there is room. If there is not room, we should NOT evict peers without pinging/dialling them.
  2. Useful peers are those that have successfully responded to a query.
@achingbrain
Copy link
Member Author

  1. is implemented by feat: ping old DHT peers before eviction #229

@achingbrain achingbrain mentioned this issue Oct 1, 2021
Closed
13 tasks
@BigLep BigLep linked a pull request Nov 13, 2021 that will close this issue
feat: ping old DHT peers before eviction #229
Merged
@SgtPooki
Copy link
Member

@achingbrain looks like we can close this?

@achingbrain
Copy link
Member Author

I don't think the second part has been done yet - adding some metadata to the routing table that says a peer has responded to a query.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: ping old DHT peers before eviction libp2p/js-libp2p-kad-dht
2 participants
@achingbrain @SgtPooki