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

Run fast catchup using peerstore instead of phonebook #5666

Open
winder opened this issue Aug 15, 2023 · 2 comments
Open

Run fast catchup using peerstore instead of phonebook #5666

winder opened this issue Aug 15, 2023 · 2 comments
Assignees
@algorandskiy
Labels
p2p Work related to the p2p project Team Lamprey

Comments

@winder
Copy link
Contributor

winder commented Aug 15, 2023

  • Initialize peerstore from SRV records / phonebook
  • Replace usages of phonebook with peerstore (or an interface that uses the peerstore)
    • i.e. peerselector, maybe others.
  • might require workarounds for peerIDs with no public key.
@winder winder added p2p Work related to the p2p project Team Lamprey labels Aug 15, 2023
@algorandskiy
Copy link
Contributor

algorandskiy commented Jan 26, 2024 •
edited

I guess this includes implementing rate limiting functionality like existing phonebookImpl has (used to rate limit outgoing HTTP queries).

@algorandskiy
Copy link
Contributor

I added more usage of phonebook methods in #5935 but the interfaces a bit diverged because of the address: string vs AddrInfo.

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

@algorandskiy algorandskiy

Labels
p2p Work related to the p2p project Team Lamprey
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@winder @algorandskiy @shiqizng