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

Support the ability for validators to validate blocks via data availability proofs #65

Closed
musalbas opened this issue Oct 6, 2020 · 2 comments
Labels
C:data-availability Component: Data Availability Proofs T:investigate Further investigation needed
Projects

Comments

@musalbas
Copy link
Member

musalbas commented Oct 6, 2020

A current limitation of Tendermint is that before a block has consensus from the validator set, it is only distributed among the validator set. This makes it not easily possible for these validators to validate the block via a data availability proof, because the minimum number of light client assumption is less likely to hold. Consider publishing blocks to the wider network before they have consensus.

@liamsi
Copy link
Member

liamsi commented Oct 6, 2020

According to @marbar3778 the block is distributed not only among the validators. Other nodes get to see the block before it has consensus too (only validators add their signatures though). This could be as simple as adding in the DA checks (and making Tx execution optional).

Related reactor is the Consensus reactor as far as I understand: https://github.com/tendermint/spec/blob/master/spec/reactors/consensus/consensus-reactor.md

@liamsi liamsi added the T:investigate Further investigation needed label Oct 6, 2020
@liamsi liamsi mentioned this issue Oct 30, 2020
6 tasks
@liamsi liamsi added the C:data-availability Component: Data Availability Proofs label Oct 31, 2020
@liamsi liamsi added this to To do in Devnet via automation Mar 17, 2021
@liamsi
Copy link
Member

liamsi commented Aug 16, 2021

We decided that validators will not do DAS. Hence, closing. (cc @musalbas @adlerjohn)

@liamsi liamsi closed this as completed Aug 16, 2021
Devnet automation moved this from To do to Done Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:data-availability Component: Data Availability Proofs T:investigate Further investigation needed
Projects
No open projects
Devnet
  
Done
Development

No branches or pull requests

2 participants