-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
planner, executor: adjustOverlongColName for CreateView (#14850) #14873
Conversation
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-all-tests |
@XuHuaiyu merge failed. |
@XuHuaiyu PTAL |
/merge |
Your auto merge job has been accepted, waiting for 15365, 14939 |
/run-all-tests |
@XuHuaiyu merge failed. |
a data race detected in etcd:
|
cherry-pick #14850
What problem does this PR solve?
Fix the bug:
What is changed and how it works?
Rename the output column name for view if the column name is too long (>64).
Check List
Tests
Code changes
Side effects
In MySQL, use the sql shown in the result of
show create view
to create a view will fail. It is really strange. In TiDB, this case will fail for the first time.case 2:
In MySQL, the following case will fail. But it will execute successfully in TiDB.
Related changes
Release note