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

Straggler handling v2.0 #4

Merged
merged 11 commits into from
Jul 22, 2024
Merged

Conversation

ParthMandaliya
Copy link
Owner

This PR includes following changes:

  • All flake8 issues resolved.
  • Teodor's review comments incorporated.
  • For cutoff time based policy after cutoff time is expired wait for all collaborators instead of minimum_reporting collaborators.
  • As soon as minimum_reporting collaborators report task results end round early regardless of tasks assigned to each collaborator.

1. In cutoff time based policy after cutoff time expires wait for all collaborators not just minimum required.
2. Irregardless of tasks assigned to collaborators if minimum required collaborators report resultsw in time apply straggler handling policy.

Signed-off-by: Parth Mandaliya <[email protected]>
Signed-off-by: Parth Mandaliya <[email protected]>
Signed-off-by: Parth Mandaliya <[email protected]>
Signed-off-by: Parth Mandaliya <[email protected]>
…ed collaborators to report results.

Signed-off-by: Parth Mandaliya <[email protected]>
…s not only for minimum_reporting collaborators

Signed-off-by: Parth Mandaliya <[email protected]>
flake8 issues resovled.

Signed-off-by: Parth Mandaliya <[email protected]>
Merged conditions in straggler_cutoff_check function in CutoffTimeBasedStragglerHandling class.

Signed-off-by: Parth Mandaliya <[email protected]>
Signed-off-by: Parth Mandaliya <[email protected]>
@ParthMandaliya ParthMandaliya merged commit b8a85c9 into straggler-handling Jul 22, 2024
1 of 2 checks passed
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.

1 participant