This repository has been archived by the owner on Jan 29, 2024. It is now read-only.
Refactor checkpoints #74
Merged
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
Co-authored-by: adlrocha <adlrocha@tutamail.com>
…c-actors into weighted_vote
adlrocha
approved these changes
Mar 22, 2023
gateway/src/checkpoint.rs
Outdated
| @@ -63,17 +63,17 @@ impl Checkpoint { | |||
| } | |||
|
|
|||
| /// return cross_msg included in the checkpoint. | |||
| pub fn cross_msgs(&self) -> Option<&CrossMsgMeta> { | |||
| pub fn cross_msgs(&self) -> &Vec<CrossMsg> { | |||
There was a problem hiding this comment.
I guess there's no need for the ref now that this is a vector.
Suggested change
| pub fn cross_msgs(&self) -> &Vec<CrossMsg> { | |
| pub fn cross_msgs(&self) -> Vec<CrossMsg> { |
gateway/src/lib.rs
Outdated
| })?; | ||
| } | ||
| let mut value = TokenAmount::zero(); | ||
| commit.cross_msgs().iter().for_each(|m| value += &m.msg.value); |
There was a problem hiding this comment.
I guess you'll accommodate this in the next PR/commit, but here we need to call apply_msg to execute the messages committed.
* update bottom up execution * update cross message execution * fix fmt * update review and clean up * check message ordering * Cross execution tests (#76) * fix clippy * fmt code
adlrocha
added a commit
that referenced
this pull request
Apr 10, 2023
* add cron fields * fmt code * Update gateway/src/state.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * Update gateway/src/types.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * update cron (#65) * update cron * fix lint --------- Co-authored-by: willesxm <willeslau@gmail.com> * add submit cron impl * add more checks * add some todo * derive total validators * add todo * specify rust tool chain * add tests * support abort * simplify impl * Track validators (#70) * track validators * add validator check to submit cron * update impl * Weighted vote (#71) * track validators * add validator check to submit cron * update impl * weighted vote * Update gateway/src/cron.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * update method name --------- Co-authored-by: adlrocha <adlrocha@tutamail.com> * Cron submit tests (#73) * track validators * add validator check to submit cron * update impl * weighted vote * Update gateway/src/cron.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * update method name * add tests * refactor pending epoches * fix clippy * add more tests --------- Co-authored-by: adlrocha <adlrocha@tutamail.com> * Refactor checkpoints (#74) * track validators * add validator check to submit cron * update impl * weighted vote * Update gateway/src/cron.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * update method name * add tests * refactor pending epoches * fix clippy * add more tests * initial commit * Cross execution (#75) * update bottom up execution * update cross message execution * fix fmt * update review and clean up * check message ordering * Cross execution tests (#76) * fix clippy * fmt code --------- Co-authored-by: adlrocha <adlrocha@tutamail.com> * cargo fmt * Vote checkpoints (#81) * track validators * add validator check to submit cron * update impl * weighted vote * Update gateway/src/cron.rs Co-authored-by: adlrocha <adlrocha@tutamail.com> * update method name * add tests * refactor pending epoches * fix clippy * add more tests * initial commit * Cross execution (#75) * update bottom up execution * update cross message execution * fix fmt * update review and clean up * check message ordering * Cross execution tests (#76) * fix clippy * fmt code * generics for cron submission * migrate to sdk * format code * remove wip field * work in progress * local changes * reorg code * update comment * update tests * format code and clippy * fix error --------- Co-authored-by: adlrocha <adlrocha@tutamail.com> * rename checkpoints to bottomup and topdown * fix tests * use CrossMsg for topdown checkpoint * fix checkpoint (#85) * update queue serialization (#86) * update queue serialization * remove println * fix fmt * genesis_epoch bottomup checkpoints to zero * fix test with new genesis checkpoint --------- Co-authored-by: willesxm <willeslau@gmail.com> Co-authored-by: adlrocha <adlrocha@tutamail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Changes
Initial implementation to simplify bottom up messages. Replaced meta with vec.