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.

Sign up for GitHub

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

Jump to bottom

Support multi-keyspace-group and sharding in tso grpc service #6232

Closed
13 tasks done
binshi-bing opened this issue Mar 27, 2023 · 1 comment
Closed
13 tasks done

Support multi-keyspace-group and sharding in tso grpc service #6232

binshi-bing opened this issue Mar 27, 2023 · 1 comment
Assignees
@JmPotato @lhy1024 @rleungx @binshi-bing
Labels
type/development The issue belongs to a development tasks

Comments

@binshi-bing
Copy link
Contributor

binshi-bing commented Mar 27, 2023 •
edited
Loading

Development Task

Issues: Ref #5895
@binshi-bing binshi-bing added the type/development The issue belongs to a development tasks label Mar 27, 2023
@binshi-bing binshi-bing mentioned this issue Mar 27, 2023
Closed
60 tasks
@binshi-bing binshi-bing changed the title Support multiple keyspace groups and sharding in TSO Support multiple keyspace groups and sharding in TSO grpc service Mar 27, 2023
@binshi-bing binshi-bing changed the title Support multiple keyspace groups and sharding in TSO grpc service Support multiple keyspace groups and sharding in tso grpc service Mar 27, 2023
@binshi-bing binshi-bing changed the title Support multiple keyspace groups and sharding in tso grpc service Support multi-keyspace-group and sharding in tso grpc service Mar 27, 2023
@binshi-bing binshi-bing mentioned this issue Mar 30, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 6, 2023
@binshi-bing
Load initial assignment and dynamically watch/apply keyspace groups' …
9fcde12 
…membership/distribution change (#6247)

ref #6232

Load initial keyspace group assignment.
Dynamically watch/apply keyspace groups' membership/distribution change.

Signed-off-by: Bin Shi <[email protected]>
This was referenced Apr 6, 2023
Merged
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 7, 2023
@binshi-bing
Add more failure tests when tso service loading initial keyspace grou…
6bac6b7 
…ps assignment (#6280)

ref #6232

Add more failure tests when tso service loading initial keyspace groups assignment

Signed-off-by: Bin Shi <[email protected]>
This was referenced Apr 7, 2023
Closed
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 11, 2023
@binshi-bing
Apply multi-keyspace-group membership to tso service and handle incon…
f80b215 
…sistency issue (#6282)

ref #6232

Apply multi-keyspace-group membership to tso service and handle inconsistency issue.

1. Add KeyspaceLookupTable to endpoint.KeyspaceGroup
type KeyspaceGroup struct {
        ...
        // KeyspaceLookupTable is for fast lookup if a given keyspace belongs to this keyspace group.
        // It's not persisted and will be built when loading from storage.
        KeyspaceLookupTable map[uint32]struct{} `json:"-"`
}

2. After loading keyspace groups, the Keyspace Group Manager builds KeyspaceLookupTable for every keyspace groups.

3. When Keyspace Group Manager handles tso requests, it uses the keyspaceLookupTable to check if the required keypsace still belongs to the required keyspace group. If not, returns the current keyspace group id in the tso response header.

Signed-off-by: Bin Shi <[email protected]>
ti-chi-bot added a commit that referenced this issue Apr 11, 2023
@JmPotato @ti-chi-bot
keyspace, api: support the keyspace group split (#6293)
3e6ac89 
ref #6232

Support the keyspace group split and add related tests.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
This was referenced Apr 11, 2023
Merged
Merged
@JmPotato JmPotato mentioned this issue Apr 12, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 12, 2023
@binshi-bing
Improve lock mechanism in tso.KeyspaceGroupManager (#6305)
f6a4090 
ref #6232

Use the RWMutex instead of individual atomic types to better protect the state of the keyspace group manager

Signed-off-by: Bin Shi <[email protected]>
@rleungx rleungx mentioned this issue Apr 12, 2023
Merged
ti-chi-bot added a commit that referenced this issue Apr 12, 2023
@JmPotato @ti-chi-bot
keyspace: add split-from field for endpoint.KeyspaceGroup (#6309)
ed13d5d 
ref #6232

Add `split-from` field for `endpoint.KeyspaceGroup`.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
ti-chi-bot added a commit that referenced this issue Apr 12, 2023
@binshi-bing @ti-chi-bot
Add read lock at one place for protection and better structure (#6310)
dc5cfeb 
ref #6232, ref #6305

follow-up #6305 
Add read lock at one place for protection and better structure

Signed-off-by: Bin Shi <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
This was referenced Apr 12, 2023
Merged
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 13, 2023
@JmPotato
tso: optimize function signatures to reduce parameter passing (#6315)
643b842 
ref #6232

Optimize function signatures to reduce parameter passing.

Signed-off-by: JmPotato <[email protected]>
@JmPotato JmPotato mentioned this issue Apr 13, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 13, 2023
@JmPotato
keyspace: avoid keyspace being updated during the split (#6316)
bac5442 
ref #6232

Prevent keyspace from being updated during the split.

Signed-off-by: JmPotato <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 8, 2023
@JmPotato @ti-chi-bot @zeminzhou
tests: enable TestTSOKeyspaceGroupSplitClient (tikv#6398)
e1aa767 
ref tikv#6232

Enable `TestTSOKeyspaceGroupSplitClient`.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 8, 2023
@JmPotato @rleungx @zeminzhou
keyspace, api2: fix the keyspace assignment patrol consistency (tikv#…
6784c9e 
…6397)

ref tikv#6232

Fix the keyspace assignment patrol consistency.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 8, 2023
@JmPotato @zeminzhou
keyspace: patrol the keyspace assignment in batch (tikv#6411)
84d9109 
ref tikv#6232

Patrol the keyspace assignment in batch.

Signed-off-by: JmPotato <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 8, 2023
@JmPotato @ti-chi-bot @zeminzhou
mcs, tso: add API interface to obtain the TSO keyspace group member i…
b5eb466 
…nfo (tikv#6373)

ref tikv#6232

Add API interface to obtain the TSO keyspace group member info.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@JmPotato JmPotato mentioned this issue May 9, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue May 10, 2023
@JmPotato @ti-chi-bot
pd-ctl, tests: add the keyspace group commands (#6423)
fa6a60c 
ref #6232

Add the keyspace group commands to show and split keyspace groups.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 10, 2023
@JmPotato @ti-chi-bot @zeminzhou
tests: enable TestTSOKeyspaceGroupSplitClient (tikv#6398)
a9b8f0b 
ref tikv#6232

Enable `TestTSOKeyspaceGroupSplitClient`.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: zeminzhou <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 10, 2023
@JmPotato @rleungx @zeminzhou
keyspace, api2: fix the keyspace assignment patrol consistency (tikv#…
6853a69 
…6397)

ref tikv#6232

Fix the keyspace assignment patrol consistency.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
Signed-off-by: zeminzhou <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 10, 2023
@JmPotato @zeminzhou
keyspace: patrol the keyspace assignment in batch (tikv#6411)
251721b 
ref tikv#6232

Patrol the keyspace assignment in batch.

Signed-off-by: JmPotato <[email protected]>
Signed-off-by: zeminzhou <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 10, 2023
@JmPotato @ti-chi-bot @zeminzhou
mcs, tso: add API interface to obtain the TSO keyspace group member i…
2b89d14 
…nfo (tikv#6373)

ref tikv#6232

Add API interface to obtain the TSO keyspace group member info.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: zeminzhou <[email protected]>
zeminzhou pushed a commit to zeminzhou/pd that referenced this issue May 10, 2023
@JmPotato @ti-chi-bot @zeminzhou
pd-ctl, tests: add the keyspace group commands (tikv#6423)
8525746 
ref tikv#6232

Add the keyspace group commands to show and split keyspace groups.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: zeminzhou <[email protected]>
@JmPotato JmPotato mentioned this issue Jun 9, 2023
Closed
This was referenced Jun 20, 2023
Merged
Merged
ti-chi-bot bot added a commit that referenced this issue Jun 21, 2023
@JmPotato @ti-chi-bot
tso: fix checkTSOSplit to finish split correctly (#6652)
4a9dcaf 
ref #6232

Fix `checkTSOSplit` to finish split correctly.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot added a commit that referenced this issue Jun 25, 2023
@JmPotato @ti-chi-bot
keyspace, apiv2: support to split keyspace group with the keyspace ID…
ec99986 
… range (#6646)

ref #6232

Support to split keyspace group with the keyspace ID range.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
tests, tso: add more TSO split tests (tikv#6338)
9363e4e 
ref tikv#6232

Add more TSO split tests.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@binshi-bing @rleungx
keyspace, tso: fix next revision to watch after watch/Get/RangeScan (t…
3e1ebae 
…ikv#6353)

ref tikv#6232

The next revision to watch should always be Header.Revision + 1 where header is response header of watch/Get/RangeScan

Signed-off-by: Bin Shi <[email protected]>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@binshi-bing @ti-chi-bot @rleungx
mcs, tso: support multi-keyspace-group and its service discovery in E…
33ec5ff 
…2E path (tikv#6321)

ref tikv#6232

Support multi-keyspace-group in PD(TSO) client

Signed-off-by: Bin Shi <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@binshi-bing @ti-chi-bot @rleungx
tso: fix bugs to make split test case to pass (tikv#6389)
4a374c6 
ref tikv#6232

fix bugs to make split test case to pass

Signed-off-by: Bin Shi <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
keyspace: patrol keyspace assignment before the first split (tikv#6388)
a1d541e 
ref tikv#6232

Patrol the keyspace assignment before the first split to make sure every keyspace has its group assignment.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
tests: enable TestTSOKeyspaceGroupSplitClient (tikv#6398)
f4bdfaa 
ref tikv#6232

Enable `TestTSOKeyspaceGroupSplitClient`.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx added a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @rleungx
keyspace, api2: fix the keyspace assignment patrol consistency (tikv#…
08f533d 
…6397)

ref tikv#6232

Fix the keyspace assignment patrol consistency.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @rleungx
keyspace: patrol the keyspace assignment in batch (tikv#6411)
2bc9949 
ref tikv#6232

Patrol the keyspace assignment in batch.

Signed-off-by: JmPotato <[email protected]>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
mcs, tso: add API interface to obtain the TSO keyspace group member i…
2f00984 
…nfo (tikv#6373)

ref tikv#6232

Add API interface to obtain the TSO keyspace group member info.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
pd-ctl, tests: add the keyspace group commands (tikv#6423)
6d79e83 
ref tikv#6232

Add the keyspace group commands to show and split keyspace groups.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
tso: fix checkTSOSplit to finish split correctly (tikv#6652)
76936a9 
ref tikv#6232

Fix `checkTSOSplit` to finish split correctly.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
keyspace, apiv2: support to split keyspace group with the keyspace ID…
54ca074 
… range (tikv#6646)

ref tikv#6232

Support to split keyspace group with the keyspace ID range.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
tso: fix checkTSOSplit to finish split correctly (tikv#6652)
e9f4726 
ref tikv#6232

Fix `checkTSOSplit` to finish split correctly.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Aug 2, 2023
@JmPotato @ti-chi-bot @rleungx
keyspace, apiv2: support to split keyspace group with the keyspace ID…
a10bc67 
… range (tikv#6646)

ref tikv#6232

Support to split keyspace group with the keyspace ID range.

Signed-off-by: JmPotato <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JmPotato JmPotato

@lhy1024 lhy1024

@rleungx rleungx

@binshi-bing binshi-bing

Labels
type/development The issue belongs to a development tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@JmPotato @lhy1024 @rleungx @binshi-bing