-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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.
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
ddl: change drop partition and truncate partition's job args to support multi partition id array (#18419) #18930
ddl: change drop partition and truncate partition's job args to support multi partition id array (#18419) #18930
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
/run-all-tests |
/run-all-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/run-check_dev |
LGTM |
@lance6716,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: ddl(slack). |
/merge |
/run-all-tests |
cherry-pick #18419 to release-4.0
Signed-off-by: AilinKid 314806019@qq.com
What problem does this PR solve?
it is prepared for cancel
add multi partition action
which is in mid stateProblem Summary: change drop partition and truncate partition's job args to support multi partition id array
What is changed and how it works?
What's Changed:
1: change drop partition and truncate partition's job args to multi id array
2: support ddl worker to handle these two action and add ids to delete range table.
Check List
Tests
Side effects
Release note
1: rolling upgrade the cluster during the drop partition DDL operation
2: degrade to old tidb if the model data has been written and the DDL operation is not finished