Optimistic Chain Clients #2555
Draft
Optimistic Chain Clients #2555
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
afck
reviewed
Oct 1, 2024
| // Update the validator with missing information, if needed. | ||
| let delivery = CrossChainMessageDelivery::NonBlocking; | ||
| self.send_chain_information(chain_id, target_block_height, delivery) | ||
| .await?; |
There was a problem hiding this comment.
Aren't we calling send_chain_update also to update the validators about a new certificate that we just created? Because then the validators definitely won't have it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Chain clients are pessimistic in general.
ChainInfoQueriesare made to ensure that chain clients are up to date with the network's stateProposal
In certain situations, like single-owner chains or when running a node service, we can assume that the chain client is up to date and act 'optimistically'.
There is a tradeoff here, lower latency for a low probability of submitting an invalid block incurring additional fees, so the client is configurable.
This PR is still a draft as I suspect there will be some further discussion around design.
Test Plan
Regressions will be caught by tests.
Release Plan