This repository was archived by the owner on Aug 26, 2024. It is now read-only.
Add rack-aware load balancing policy #1
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We need to prefer local rack as there are higher network costs when associated with nodes in remote rack. This is initial version, some things are still missing: * no tests yet * fallback to racks in local DC is not implemented yet (i.e. this version can connect to remote DC even if there is another rack in the local DC that we haven't tried yet)
I forgot to uncomment this when debugging build yesteday.
We want to try local rack, local dc and then remote dcs, in that order.
There is no reason why not to try to use a replica first.
|
Supeseded by #2 |
|
Is this updated documented? it should! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We need to prefer local rack as there are higher network costs when associated with nodes in remote rack.
This is initial version, some things are still missing: