-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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: add owner_id field to tidb_mdl_info to avoid unexpected writes (#53234) #53300
ddl: add owner_id field to tidb_mdl_info to avoid unexpected writes (#53234) #53300
Conversation
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release-8.1 #53300 +/- ##
================================================
Coverage ? 71.1782%
================================================
Files ? 1465
Lines ? 421230
Branches ? 0
================================================
Hits ? 299824
Misses ? 100940
Partials ? 20466
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: easonn7, tangenta, YuJuncen The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This is an automated cherry-pick of #53234
What problem does this PR solve?
Issue Number: close #53073
Problem Summary:
tidb-1 keeps printing "someone is not synced", which means it is blocked at
waitSchemaSynced
.tidb-1 is DDL owner. However, tidb-1's schema version is less than non-owner TiDB (here are the results from
tiup ctl:v8.0.0 etcd --endpoints={pd_addr}:2379 get --prefix /tidb/ddl
):After checking the log, I found that DDL owner is switched from tidb-0 to tidb-1 at this time.
On the other hand,
mysql.tidb_mdl_info
is empty.Above all, the timeline should be:
At step 6, tidb-0 incorrectly clean the mdl info registered by tidb-1. As a result, there is nothing to update in
mdlCheckLoop
becausetidb_mdl_info
is empty.What changed and how does it work?
owner_id
field totidb_mdl_info
to identify which owner is used.owner_id
in WHERE clause.Check List
Tests
With this PR, we can pass the test mentioned in issue add index blocked due to tidb schema version can not sync and caused that other ddl job always queueing after injection pd leader io delay #53073.
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.