Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

share: EDSStore.Blockstore #1115

Closed
Tracked by #1099
Wondertan opened this issue Sep 15, 2022 · 0 comments · Fixed by #1395
Closed
Tracked by #1099

share: EDSStore.Blockstore #1115

Wondertan opened this issue Sep 15, 2022 · 0 comments · Fixed by #1395
Assignees
Labels
area:shares Shares and samples

Comments

@Wondertan
Copy link
Member

Wondertan commented Sep 15, 2022

Context

Blockstore method returns a Blockstore interface implementation instance, providing random access over share and NMT Merkle proof in every stored EDS. It is required for FNs/BNs to serve DAS requests over the Bitswap and for reading data by namespace.

Implementation Details

There is a frozen/un-merged implementation of Blockstore over DAGStore and CARv2 indexes. EDIT: Already unfrozen and almost ready to be merged

NOTES:

// Blockstore returns an IPFS Blockstore providing access to individual shares/nodes of all EDS 
// registered on the Store. NOTE: The Blockstore does not store whole Celestia Blocks but IPFS blocks. 
// We represent `shares` and NMT Merkle proofs as IPFS blocks and IPLD nodes so Bitswap can access those.
func (s *Store) Blockstore() blockstore.Blockstore
@Wondertan Wondertan added the area:shares Shares and samples label Sep 15, 2022
@distractedm1nd distractedm1nd self-assigned this Nov 9, 2022
distractedm1nd added a commit that referenced this issue Dec 7, 2022
renaynay pushed a commit to renaynay/celestia-node that referenced this issue Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:shares Shares and samples
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants