Skip to content

Support per-member ReplicationFactor - #30

Open
park-bit wants to merge 1 commit into
buraksezer:masterfrom
park-bit:master
Open

Support per-member ReplicationFactor#30
park-bit wants to merge 1 commit into
buraksezer:masterfrom
park-bit:master

Conversation

@park-bit

Copy link
Copy Markdown

Closes #26

so rn every member just uses the same ReplicationFactor from Config,
no way to give a bigger/heavier node more of the ring than a small one.

added a small optional interface for this:

type ReplicationFactorMember interface {
Member
ReplicationFactor() int
}

if a member implements it, Add/Remove will use its own factor instead
of the global one. if it doesn't, nothing changes, falls back to
config.ReplicationFactor like before. so shouldn't break anything.

also added a test for it, checks the vnode count matches what we'd
expect and that Remove actually cleans up all of them after.

ran everything with -race too, all passing on my end

@buraksezer
buraksezer self-requested a review August 20, 2026 22:26
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.

Support variable ReplicationFactor for Members

1 participant