lighthouse icon indicating copy to clipboard operation
lighthouse copied to clipboard

Remove default target peer for proposer-only mode for PeerDAS.

Open jimmygchen opened this issue 1 year ago • 2 comments

Issue Addressed

The default target peer for proposer node (=15) will need to be increased for peerdas. A node would need peers across all data column subnets to be able to publish the data columns successfully.

If we can have "decentralized blob building" working reliably on PeerDAS in the future, then proposers won't necessary need to publish blobs themselves and this number can potentially be reduced again.

jimmygchen avatar Jul 24 '24 05:07 jimmygchen

Please update to point at unstable by either:

  1. Rebasing on unstable (if your branch has a small number of commits that are easy to tease out), or
  2. Merging origin/das into this PR: git fetch origin; git merge origin/das. This will result in the smallest number of conflict resolutions and is better for branches that already contain merge commits or have extensive history.

michaelsproul avatar Aug 27 '24 05:08 michaelsproul

This feature makes the proposer-only mode less useful than it was. I'm inclined to see whether decentralized blob building #6268 formally make it into the spec before deciding to merge this one.

IMO I think we should ship peerdas with #6268, and we'd be able to keep the proposer-only feature.

jimmygchen avatar Aug 28 '24 06:08 jimmygchen

This PR was closed because the source branch was delete. With getBlobsV1 being included in the spec, I think we can close this anyway.

jimmygchen avatar Jan 24 '25 20:01 jimmygchen