-
Notifications
You must be signed in to change notification settings - Fork 429
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
Bundle Inbox#checkpoint with message processing #346
Comments
This was referenced May 16, 2022
Noting that we've agreed to remove the caching of checkpoints on the Inbox entirely |
Played around with some gas profiling in asaj/inbox-checkpoints You can see the results here: |
Satisfied by #523 |
Repository owner
moved this from Sprint Backlog
to Done
in Hyperlane Tasks
Jun 8, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As per the recent decision to not subsidize system processing, applications will need to pay for inbox checkpointing on the destination chain. this ticket emcompasses the following work (feel free to change the names):
InboxValidatorManager#checkpointAndProcess
which (after quorum check) basically just calls the existingInboxValidatorManager#checkpoint
function and thenInbox.process
to save the intrinsic gas costAdditionally:
onlyValidatorManager
function toInbox
calleddirectProcess
which has the same signature asprocess
except it also takes a root as an argument and then verifies the merkle proof against that (instead of a stored checkpoint)InboxValidatorManager#directProcess
which (after quorum check) callsdirectProcess
on the batch of passed in messagesActually, one should go one step further and not even store the checkpoint in storage at all, instead verify the signatures for a checkpoint and then process messages directly against it.
This ticket now exclusively reflects the scope to add these latency/gas saving combinations of checkpointing/message processing to the contracts and creates #472 for the agent specific changes
The text was updated successfully, but these errors were encountered: