-
Notifications
You must be signed in to change notification settings - Fork 261
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
v1.18: pings received contact-infos on gossip socket address (backport of #1615) #1635
Conversation
(cherry picked from commit 329a186) # Conflicts: # gossip/src/cluster_info.rs # gossip/src/legacy_contact_info.rs
Cherry-pick of 329a186 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! Thanks Behzad!
…t of anza-xyz#1615) (anza-xyz#1635) * pings received contact-infos on gossip socket address (anza-xyz#1615) (cherry picked from commit 329a186) # Conflicts: # gossip/src/cluster_info.rs # gossip/src/legacy_contact_info.rs * resolves merge conflicts --------- Co-authored-by: behzad nouri <behzadnouri@gmail.com>
Problem
bandwidth wasted on useless contact-infos.
Summary of Changes
ping received contact-infos on gossip socket address
This is an automatic backport of pull request #1615 done by Mergify.