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

Commitment of Row Commitments Only in Block Headers #38

Open
DarkingLee opened this issue Jan 4, 2024 · 0 comments
Open

Commitment of Row Commitments Only in Block Headers #38

DarkingLee opened this issue Jan 4, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@DarkingLee
Copy link
Member

The current block headers contain commitments for all rows as well as the index for App_id, leading to excessively bulky block headers. In a fully loaded state, the commitments alone can reach up to 128kb. Moreover, after sharding, we must include "commitments only". We should aim to include only the commitment of row commitments and the number of rows in the block headers. This approach requires hashing all row commitments, thereby losing homomorphic properties, but this is inconsequential in our current context.

This change will primarily affect light clients and not validators, as validators can access the actual commitments from storage. Hence, the actual data is more inclined towards using DHT (Distributed Hash Table) rather than Gossip.

@DarkingLee DarkingLee added the enhancement New feature or request label Jan 4, 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
None yet
Development

No branches or pull requests

1 participant