Skip to content

Conversation

@asmaastarkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

asmaastarkware commented Dec 8, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from b767e23 to 9eed0d4 Compare December 8, 2025 10:13
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from d1e5df5 to 5884ac0 Compare December 8, 2025 13:14
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from 9eed0d4 to c38161e Compare December 8, 2025 13:14
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from 5884ac0 to f923141 Compare December 8, 2025 13:51
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from c38161e to 4bbf79f Compare December 8, 2025 13:51
Copy link
Collaborator

@matanl-starkware matanl-starkware left a 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: :shipit: 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:

.0

@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from f923141 to 4050971 Compare December 8, 2025 14:43
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from 4bbf79f to 1f854b1 Compare December 8, 2025 14:43
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from 4050971 to e5d0769 Compare December 9, 2025 05:59
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch 2 times, most recently from 18ad804 to 9607992 Compare December 9, 2025 06:40
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from e5d0769 to 30bc6f9 Compare December 9, 2025 06:40
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from 9607992 to 99d8789 Compare December 9, 2025 08:50
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from 30bc6f9 to 383c26e Compare December 9, 2025 08:50
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/move_caching_decision_logic_to_consensuscache branch from 99d8789 to 832d59e Compare December 9, 2025 11:47
@asmaastarkware asmaastarkware force-pushed the asmaa/refactor/rename_proposal_cache_and_stream_fields branch from 383c26e to 48f32d8 Compare December 9, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants