-
Notifications
You must be signed in to change notification settings - Fork 65
apollo_consensus: move caching decision logic to ConsensusCache #10620
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
base: asmaa/refactor/rename_proposal_cache_and_stream_fields
Are you sure you want to change the base?
Conversation
b767e23 to
9eed0d4
Compare
d1e5df5 to
5884ac0
Compare
9eed0d4 to
c38161e
Compare
5884ac0 to
f923141
Compare
c38161e to
4bbf79f
Compare
matanl-starkware
left a comment
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.
@matanl-starkware reviewed 1 of 1 files at r1, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @dafnamatsry)
crates/apollo_consensus/src/manager.rs line 293 at r1 (raw file):
) -> bool { let limits = &self.future_msg_limit; let height_diff = msg_height.saturating_sub(current_height.0);
I don't think you need .0 when you derive Deref
Code quote:
.0f923141 to
4050971
Compare
4bbf79f to
1f854b1
Compare
4050971 to
e5d0769
Compare
18ad804 to
9607992
Compare
e5d0769 to
30bc6f9
Compare
9607992 to
99d8789
Compare
30bc6f9 to
383c26e
Compare
99d8789 to
832d59e
Compare
383c26e to
48f32d8
Compare

No description provided.