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

Move to fully connected network #12984

Closed

Tracked by

kfa-aguda opened this issue Apr 24, 2024 · 0 comments · Fixed by #13010

Closed

Tracked by

Move to fully connected network #12984

kfa-aguda opened this issue Apr 24, 2024 · 0 comments · Fixed by #13010

Assignees

Milestone

Contributor

kfa-aguda commented Apr 24, 2024 •

edited

This issue was formerly titled "Eliminate addressBook from Network component" with the following description

eliminate the dependency of network on addressBook
change peers list to peers set to reflect node uniqueness property. This also helps with making peer lookups O(1); an important prep for Dynamic Address Book which will be lookup heavy.

Upon subsequent internal conversations, the issue has been renamed to capture the bigger task which is:

stop the use of RandomGraph
do away with address book node indices and use node Ids
Make the network fully connected, do not support partial connectivity.

kfa-aguda mentioned this issue

Gossip for Componentization #11505

Open

kfa-aguda self-assigned this

kfa-aguda mentioned this issue

feat: Move to fully connected network #13010

Merged

kfa-aguda changed the title ~~Eliminate addressBook from Network component~~ Move to fully connected network

kfa-aguda added this to the v0.52 milestone

kfa-aguda closed this as completed in #13010

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