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.

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

Bump libp2p (GossipSub improvements) #3112

Merged
merged 1 commit into from
Nov 18, 2021
Merged

Bump libp2p (GossipSub improvements) #3112

merged 1 commit into from
Nov 18, 2021

Conversation

Menduist
Copy link
Contributor

@Menduist Menduist commented Nov 18, 2021

Includes the latest libp2p unstable merge: vacp2p/nim-libp2p#651

Relevant for nimbus:
vacp2p/nim-libp2p#634: allow applications to tune the maximum GossipSub message size, and switch default to 1Mb

vacp2p/nim-libp2p#648: Fix an issue where we relayed GossipSub messages to the peer who sent it to us. This reduce the number of message sent by about ~10%

Also, we'll avoid relaying messages to peers who sent the message while we are validating it. Currently, with validation in nimbus being near instantaneous, the effect is negligible on beefy servers (about 1% reduction in sent messages). But on more restricted nodes, it will have a bigger impact.

More importantly, this opens the opportunity to optimize the uplink traffic (and replication level in the network) even more, by waiting longer during validation

@arnetheduck arnetheduck merged commit 4bcdcca into unstable Nov 18, 2021
@arnetheduck arnetheduck deleted the bump_libp2p branch November 18, 2021 09:57
@github-actions
Copy link

github-actions bot commented Nov 18, 2021

Unit Test Results

     12 files  ±0     744 suites  ±0   33m 35s ⏱️ +45s
1 464 tests ±0  1 462 ✔️ ±0  2 💤 ±0  0 ±0 
8 924 runs  ±0  8 916 ✔️ ±0  8 💤 ±0  0 ±0 

Results for commit 14f0e8d. ± Comparison against base commit 9e39501.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants