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

Disable Peer Exchange in PRUNE message #7894

Conversation

StefanBratanov
Copy link
Contributor

@StefanBratanov StefanBratanov commented Jan 18, 2024

PR Description

Update jvm-libp2p to 1.1.0-RELEASE and set maxPeersSentInPruneMsg and maxPeersAcceptedInPruneMsg

Fixed Issue(s)

fixes #7860

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

Comment on lines +76 to +77
.maxPeersSentInPruneMsg(0)
.maxPeersAcceptedInPruneMsg(Integer.MAX_VALUE)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@Nashatyrev Nashatyrev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StefanBratanov StefanBratanov force-pushed the disable_peer_exchange_in_prune_messages branch from 3c02c5f to 7442559 Compare January 23, 2024 19:15
@StefanBratanov StefanBratanov enabled auto-merge (squash) January 23, 2024 19:16
@StefanBratanov StefanBratanov merged commit d43ba28 into Consensys:master Jan 23, 2024
14 of 15 checks passed
@StefanBratanov StefanBratanov deleted the disable_peer_exchange_in_prune_messages branch January 24, 2024 14:44
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.

Disable Peer Exchange in PRUNE message
2 participants