Closed ou-bing closed 3 years ago
@zimulala Here is the new PR for multi-schema change.
@tisonkun should we cherry-pick this to release-{4.0, 5.0, 5.1}?
@kennytm I'm not the person to make decision. May @tiancaiamao and you as the leader of sql-infra conclude it.
For suggestion, I don't see even the background as an issue or other link for this PR so that it is hard to say whether pick or not.
Sorry, wrong issue link, meant to comment on #1247 😅 @tisonkun
[REVIEW NOTIFICATION]
This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer
in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer
in the comment to help you merge this pull request.
The full list of commands accepted by this bot can be found here.
/merge
This pull request has been accepted and is ready to merge.
@ou-bing: Your PR was out of date, I have automatically updated it for you.
What problem does this PR solve?
Support to delete multiple indexes in one statement.
What is changed and how it works?
Add the new field to the Job Struct, I want to save some warnings when performing the change multi-schema operation.