-
Notifications
You must be signed in to change notification settings - Fork 933
Closed
Labels
fuluRequired for the upcoming Fulu hard forkRequired for the upcoming Fulu hard forkoptimizationSomething to make Lighthouse run more efficiently.Something to make Lighthouse run more efficiently.
Description
We've recently introduced gossip verification to getBlobsV2 response (#7493) to make sure
- The proofs from the EL are validated
- We don't re-process already imported data columns (
GossipDataColumnError::PriorKnownandGossipDataColumnError::PriorKnownUnpublished)
We currently do this by performing full gossip verification here:
https://github.com/jimmygchen/lighthouse/blob/e33dadbf1a520e92ddec614f3b7133fcc9e4c572/beacon_node/beacon_chain/src/fetch_blobs/mod.rs#L340-L352
The full verification includes relatively expensive checks like signature verification, which isn't necessary since the block headers are already verified and we're just cloning it over to build DataColumnSidecars. This impacts the block attestable time and import time. We could optimise this to have a verification mode that skips all validation on the SignedBeaconBlockHeader and still do the rest.
Metadata
Metadata
Assignees
Labels
fuluRequired for the upcoming Fulu hard forkRequired for the upcoming Fulu hard forkoptimizationSomething to make Lighthouse run more efficiently.Something to make Lighthouse run more efficiently.