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

[Feature Request]: Make Avail ID a tuple of (block_height, extrinsic_index) #39

Open
0xRampey opened this issue May 6, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@0xRampey
Copy link

0xRampey commented May 6, 2024

Implementation ideas

Currently, the da.ID implementation for AvailDA contains only the block height where a blob was posted. This is not an exact pointer to the blob and hence, the Get(ctx context.Context, ids []da.ID, namespace da.Namespace) method has to comb through all the DataTransactions in the block to find it. In fact, it looks like the Get method is currently returning all the blobs at the ID-specified block height.

Adding TransactionIndex to the ID implementation of an Avail blob should solve the above as it is unique per blob submission transaction and can be used to directly pinpoint and retrieve a blob with a specifc ID on Avail.

@0xRampey 0xRampey added the enhancement New feature or request label May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

1 participant