lighthouse icon indicating copy to clipboard operation
lighthouse copied to clipboard

Add `DataColumnSidecarsByRoot ` req/resp protocol

Open jimmygchen opened this issue 1 year ago • 0 comments

Issue Addressed

Part of #4983.

Proposed Changes

Add implementation for DataColumnSidecarsByRoot.

Currently blobs are served from three sources, in the following order

  1. data availability checker (before we have all the blocks and blobs)
  2. early attester cache (before we import the block)
  3. blobs db (after import)

I've implemented the equivalent for data columns, and I'm reusing the blobs db for simplicity now.

This PR does not cover the following:

  1. Sending outbound requests
  2. Handling of responses from peers

Remaining work:

  • [ ] Review logic for storing data columns (ideally make it work pre/post EIP-7594)
  • [ ] Fix failing tests
  • [ ] Add some test cases for data column

jimmygchen avatar Feb 05 '24 23:02 jimmygchen