Skip to content
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

#3933 DrbComputations helper for quorum_vote #3941

Merged
merged 10 commits into from
Dec 4, 2024
Merged

#3933 DrbComputations helper for quorum_vote #3941

merged 10 commits into from
Dec 4, 2024

Conversation

pls148
Copy link
Contributor

@pls148 pls148 commented Dec 3, 2024

Closes #3933

This PR:

Moves the logic for DRB computation tasks into a helper struct

This PR does not:

Key places to review:

@pls148 pls148 requested review from shenkeyao and ss-es December 3, 2024 21:16
@pls148 pls148 requested a review from bfish713 as a code owner December 3, 2024 21:16
@pls148 pls148 force-pushed the ps/3933 branch 2 times, most recently from a9adcc4 to 9257bdc Compare December 3, 2024 21:33
@pls148 pls148 force-pushed the ps/3933 branch 2 times, most recently from df6424a to 05fff02 Compare December 3, 2024 23:51
crates/task-impls/src/quorum_vote/handlers.rs Outdated Show resolved Hide resolved
crates/task-impls/src/quorum_vote/handlers.rs Outdated Show resolved Hide resolved
crates/task-impls/src/quorum_vote/handlers.rs Show resolved Hide resolved
crates/task-impls/src/quorum_vote/handlers.rs Outdated Show resolved Hide resolved
Copy link
Member

@shenkeyao shenkeyao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nit comment.

crates/task-impls/src/quorum_vote/drb_computations.rs Outdated Show resolved Hide resolved
@pls148 pls148 merged commit 2292e5d into main Dec 4, 2024
17 checks passed
@pls148 pls148 deleted the ps/3933 branch December 4, 2024 23:44
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.

[DRB] - Update the computation map
4 participants