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

Simplify the balancer interface, so it only deals with claiming tasks. #83

Open
epsniff opened this issue Dec 23, 2014 · 0 comments
Open

Comments

@epsniff
Copy link
Contributor

epsniff commented Dec 23, 2014

We are considering simplifying the Balancer interface to be just a Claimer function. And the balancer code will be its own function, since in practice we only have one implementation of balancer but many types of claimers.

A possible interface could be: type Claimer func(ctx BalancerContext, task string) bool This could also make the decorator pattern even easier to use with Claiming tasks.

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

No branches or pull requests

1 participant