feat(ring): Add the option of having a lower consistency level#686
Draft
mbarbon wants to merge 4 commits intografana:mainfrom
Draft
feat(ring): Add the option of having a lower consistency level#686mbarbon wants to merge 4 commits intografana:mainfrom
mbarbon wants to merge 4 commits intografana:mainfrom
Conversation
This is consistent with what defaultReplicationStrategy is doing, and allows for read consistency different from quorum.
- any always reads from a single zone - relaxed_quorum tries to use a quorum, and if that fails uses all available zones Enabling this setting trades consistency for availability.
|
|
4 tasks
Contributor
|
Hi! 👋 We recently merged #947 which changed the CI configuration for unit tests. To get CI passing on this PR, please rebase on the latest Thanks! |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This set of changes adds an optional consistency level for
ring.Ring(defaulting to existing behavior of a quorum ofRF/2+1).This setting is going to be used by a follow-up change in Mimir (draft here) to optionally use a lower consistency level in read operations to trade lower consistency for higher availability.
Checklist
CHANGELOG.mdupdated - the order of entries should be[CHANGE],[FEATURE],[ENHANCEMENT],[BUGFIX]