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.

Sign up for GitHub

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

Jump to bottom

Cleanup FairBalancer.CanClaim #125

Open
schmichael opened this issue May 21, 2015 · 0 comments
Open

Cleanup FairBalancer.CanClaim #125

schmichael opened this issue May 21, 2015 · 0 comments
Labels
enhancement performance

Comments

@schmichael
Copy link
Contributor

PR #123 added really naive logic to delay task claiming for the FairBalancer because recent changes made it so overloaded nodes could happily keep winning claim races.

The hack works, but there's got to be a better way. Adding an integration test around etcd+FairBalancer would probably be helpful.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@schmichael