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
runtime/consensus/tendermint/verifier: Support same-block validation #5300
Conversation
d640e69
to
ebf412c
Compare
if err != nil { | ||
return nil, fmt.Errorf("client: failed to get epoch at height %d: %w", annBlk.Height, err) | ||
} | ||
// TODO: Remove once same block consensus validation is deployed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created a task for this so we don't forget.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will create a draft PR also.
Yeah, makes sense, will open an issue. |
ebf412c
to
0b29088
Compare
The post-execution state of the latest consensus block is now verified using the block metadata transaction, effectively eliminating any block delay for state verification.
0b29088
to
167d666
Compare
Using metadata transaction to verify the latest consensus block. These changes should allow queries for the latest runtime state.
Currently, light blocks are fetched twice for the same height, i.e. before sending light block to the runtime, and when doing runtime header verification. This could be optimized by sending height instead of a block.