apache / apache/kvrocks

support transcation processing for `cluster setslot`

Open
#1,439 4 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
4.4k
Forks
658
Avg merge
1d 20h
Merged PRs (30d)
10

Description

### Search before asking

- [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.

### Motivation

_Originally posted by @infdahai in https://github.com/apache/incubator-kvrocks/pull/1414#discussion_r1185736226_

`Cluster::SetSlotRanges` exists consistency issues if `slot` in the middle is down
https://github.com/apache/incubator-kvrocks/blob/d2d10f7dd220992a2c7cf9f1b44a96a6c6ac80a7/src/cluster/cluster.cc#L123-L145

### Solution

@caipengbo

### Are you willing to submit a PR?

- [ ] I'm willing to submit a PR!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with Cluster::SetSlotRanges in src/cluster/cluster.cc, especially the linked lines 123-145, and review the discussion in pull request #1414. Determine the transaction-processing behavior required for cluster setslot when a middle slot is down; done means the slot-range update remains consistent in that failure case.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, redis
Domain
databases, distributed-systems
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.