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

Split state cannot recover if there is no request #6696

Closed
rleungx opened this issue Jun 27, 2023 · 1 comment · Fixed by #6736
Closed

Split state cannot recover if there is no request #6696

rleungx opened this issue Jun 27, 2023 · 1 comment · Fixed by #6736
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@rleungx
Copy link
Member

rleungx commented Jun 27, 2023 •
edited
Loading

Enhancement

If the split target group doesn't receive the request, it will stay in the split state, which cannot be selected as the split source.
@rleungx rleungx added the type/bug The issue is confirmed as a bug. label Jun 27, 2023
@lhy1024 lhy1024 mentioned this issue Jun 27, 2023
Closed
@lhy1024
Copy link
Contributor

lhy1024 commented Jun 27, 2023

tso-0.log
tso-1.log

@rleungx rleungx added type/enhancement The issue or PR belongs to an enhancement. and removed type/bug The issue is confirmed as a bug. labels Jun 28, 2023
@rleungx rleungx changed the title Split state cannot recover if the instance crushes in split process Split state cannot recover if there is no request Jun 28, 2023
@lhy1024 lhy1024 mentioned this issue Jul 4, 2023
Merged
@ti-chi-bot ti-chi-bot bot closed this as completed in #6736 Jul 4, 2023
ti-chi-bot bot added a commit that referenced this issue Jul 4, 2023
@JmPotato @ti-chi-bot
tso: implement groupSplitPatroller to speed up the split process (#6736)
d0c03a3 
ref #5895, close #6696

Implement `groupSplitPatroller` to speed up the split process.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

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: implement groupSplitPatroller to speed up the split process (tik…
3d8f8d1 
…v#6736)

ref tikv#5895, close tikv#6696

Implement `groupSplitPatroller` to speed up the split process.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

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: implement groupSplitPatroller to speed up the split process (tik…
837661b 
…v#6736)

ref tikv#5895, close tikv#6696

Implement `groupSplitPatroller` to speed up the split process.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

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
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tso: implement groupSplitPatroller to speed up the split process JmPotato/pd
2 participants
@lhy1024 @rleungx