Remove peer sampling code#7768
Conversation
pawanjay176
left a comment
There was a problem hiding this comment.
LGTM, love removing code.
Just want to run a sync test to make sure nothing depended on sampling
Good idea, triggered one now |
|
@pawanjay176 sync tests are working - except for Fulu backfill for superdnodes which still needs retry implemented in your PR. I'll go ahead and merge this one. |
|
This pull request has been removed from the queue for the following reason: The merge conditions cannot be satisfied due to failing checks: You may have to fix your CI before adding the pull request to the queue again. |
Issue Addressed
Peer sampling has been completely removed from the spec. This PR removes our partial implementation from the codebase.
ethereum/consensus-specs#4393