Skip to content

Comments

Sync peer attribution#7726

Closed
pawanjay176 wants to merge 11 commits intosigp:fusaka-devnet-2from
pawanjay176:sync-peer-attribution
Closed

Sync peer attribution#7726
pawanjay176 wants to merge 11 commits intosigp:fusaka-devnet-2from
pawanjay176:sync-peer-attribution

Conversation

@pawanjay176
Copy link
Member

Issue Addressed

N/A

Proposed Changes

Improvements to range sync including:

  1. Contain column requests only to peers that are part of the SyncingChain
  2. Attribute the fault to the correct peer and downscore them if they don't return the data columns for the request
  3. Improve sync performance by retrying only the failed columns from other peers instead of failing the entire batch
  4. Uses the earliest_available_slot to make requests to peers that claim to have the epoch. Note: if no earliest_available_slot info is available, fallback to using previous logic i.e. assume peer has everything backfilled upto WS checkpoint/da boundary

Tested this on fusaka-devnet-2 with a full node and supernode and the recovering logic seems to works well.
Also tested this a little on mainnet.

Need to do more testing and possibly add some unit tests.

@pawanjay176 pawanjay176 requested a review from jxs as a code owner July 10, 2025 03:13
@mergify mergify bot deleted the branch sigp:fusaka-devnet-2 July 10, 2025 21:33
@mergify mergify bot closed this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants