-
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
lightning: don't ignore error if check new collation failed #31089
Conversation
[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 The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Code Coverage Details: https://codecov.io/github/pingcap/tidb/commit/4ca5fa1e5e0862a1dbce13beaa8d9fa9e92a8e99 |
@@ -379,9 +379,13 @@ func ObtainNewCollationEnabled(ctx context.Context, g glue.SQLExecutor) bool { | |||
) | |||
if err == nil && newCollationVal == "True" { | |||
newCollationEnabled = true | |||
} else if errors.ErrorEqual(err, sql.ErrNoRows) { | |||
// ignore if target variable is not found, this may happen if tidb < v4.0 | |||
newCollationEnabled = false |
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.
No need to set it.
/merge |
This pull request has been accepted and is ready to merge. Commit hash: fc19f58
|
Please follow PR Title Format:
Or if the count of mainly changed packages are more than 3, use
After you have format title, you can leave a comment |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-5.2 in PR #31151 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-5.3 in PR #31152 |
What problem does this PR solve?
Issue Number: close #31088
Problem Summary:
What is changed and how it works?
Do not ignore the error when execute query
SELECT variable_value FROM mysql.tidb WHERE variable_name = 'new_collation_enabled'
, if the return err issql.NORows
, lightning will treat it as false, otherwise, directly return an error and exit.Check List
Tests
Side effects
Documentation
Release note