Jimmy Chen
Jimmy Chen
I think we should review blob publishing before merging this to avoid adding excessive bandwidth usage for Deneb.
Be great to have https://github.com/sigp/lighthouse/pull/6403 merged first, as there will be some more conflicts that needs to be resolved, and a bit easier in this order. We also need to...
Remaining TODOs - [x] prioritise block publishing (https://github.com/sigp/lighthouse/pull/6268/commits/0a2c6f76c790e2117ed4a225ef312795eb26f240) - [x] send `IDONTWANT` on publish - #6513
I'm observing a few things from testing: The node sometimes end up publishing all blobs, if the pending component is not found in availability cache after the block has been...
@dapplion @michaelsproul I've made a few more changes and have tested this locally. This is ready for review now: - Fix invalid inclusion proof for EL blobs - Only publish...
Some findings from mainnet testing from *a small sample size*: - EL `getBlobs` only gets triggered once roughly every ~5-10 mins. This is likely because relays publish blobs before blocks....
One more thing I was thinking about - we may not want to _always_ fetch blobs on rpc response, as it only makes sense / works if we're close to...
A few other clients have implemented this, and only serve the request if the node already have at least 50% of columns. Fetching 50% / 64 columns from peers isn't...
Completed in #6755
The peer selection for custody column peer happens here: https://github.com/sigp/lighthouse/blob/90700fe12a28d2795d81a9cbb23cd2b19308d96a/beacon_node/lighthouse_network/src/peer_manager/peerdb.rs#L259-L278