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

Route balancing using jumping consistent hashing algorithm (#2215) #2401

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pdorofiejczyk
Copy link
Contributor

  1. Added new balancing algorithm (detailed explanation in Route selection/balancing algorithm #2215)
  2. Added configuration option domain_balancing_algorithm with possible values: dynamic, fix_number and consistent_hashing where:
  • dynamic behaves like in situation when domain_balancing_component_number has not been set
  • fix_number behaves like in situation when domain_balancing_component_number has been set
  • consistent_hashing is the new algorithm explained in Route selection/balancing algorithm #2215
  1. When domain_balancing_algorithm is not set, fallback to domain_balancing_component_number logic is used

@cromain cromain requested a review from zinid April 30, 2018 15:04
@p1bot
Copy link
Collaborator

p1bot commented Jan 21, 2019

Hi @pdorofiejczyk, many thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@p1bot p1bot added the cla-missing Contributor needs to sign Contribution License Agreement label Jan 21, 2019
@p1bot
Copy link
Collaborator

p1bot commented Mar 11, 2019

You did it @pdorofiejczyk!

Thank you for signing the ProcessOne Contribution License Agreement.

We will have a look at your contribution!

@p1bot p1bot removed the cla-missing Contributor needs to sign Contribution License Agreement label Mar 11, 2019
@Neustradamus
Copy link

@pdorofiejczyk: Can you rebase your PR, it will be nice to see this improvement merged...

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