ddl: return early when encounter error in BackendCtx.Unregister() (#43326)#43330
Conversation
|
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. DetailsReviewer can indicate their review by submitting an approval review. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hawkingrei, zimulala The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
This is an automated cherry-pick of #43326
What problem does this PR solve?
Issue Number: close #43323
Problem Summary:
DROP INDEXas a rollback job.convertAddIdxJob2RollbackJob,cleanupLocalIndexData()is invoked to remove the lightning sort path.BackendCtx.Unregister(), TiDB fails to close the engine because the sort path is removed.delete-only, however the job is statecancelled.What is changed and how it works?
To address step 2 and step 4, this PR
cleanupLocalIndexData()because the sort path removing is considered byUnregister().BackendCtx.Unregister()Check List
Tests
ADD INDEXjob successfully.Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.