Skip to content

Commit

Permalink
fix(collator): use lt from Account for groups
Browse files Browse the repository at this point in the history
  • Loading branch information
Rexagon committed Jun 14, 2024
1 parent 76dafeb commit 308d747
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 7 additions & 6 deletions collator/src/collator/execution_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -141,17 +141,16 @@ impl ExecutionManager {

let executor_output = tx.result?;

self.min_next_lt =
cmp::max(self.min_next_lt, executor_output.account_last_trans_lt);

items.push(ExecutedTickItem {
account_addr: executed.account_state.account_addr,
in_message: tx.in_message,
executor_output,
});
}

self.min_next_lt = cmp::max(
self.min_next_lt,
executed.account_state.shard_account.last_trans_lt + 1,
);
self.accounts_cache
.add_account_stuff(executed.account_state);
}
Expand Down Expand Up @@ -216,7 +215,9 @@ impl ExecutionManager {
})
.await?;

self.min_next_lt = cmp::max(min_next_lt, account_stuff.shard_account.last_trans_lt + 1);
if let Ok(executor_output) = &executed.result {
self.min_next_lt = cmp::max(min_next_lt, executor_output.account_last_trans_lt);
}
self.accounts_cache.add_account_stuff(account_stuff);
Ok(executed)
}
Expand Down Expand Up @@ -245,7 +246,7 @@ impl ExecutionManager {
})
.await?;

self.min_next_lt = cmp::max(min_next_lt, account_stuff.shard_account.last_trans_lt + 1);
self.min_next_lt = cmp::max(min_next_lt, executed.account_last_trans_lt);
self.accounts_cache.add_account_stuff(account_stuff);
Ok(executed)
}
Expand Down

0 comments on commit 308d747

Please sign in to comment.