-
Notifications
You must be signed in to change notification settings - Fork 489
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
Fix TiDB behavior: with validation #606
Comments
Should these problem about incompatible statements be fixed in repo parser or repo tidb? |
@yuanjize We need to update the code in TiDB repo. |
@yuanjize If you have any questions, feel free to ask here or contact me on slack :).
|
@tangenta , I'm working on this issue as a new contributor. A warning is threw when I execute
Has this issue been solved already? |
@cncal This issue has not been fixed yet. Your contribution is welcomed :) |
I think the warning has been generated by parser, should I remove the code that generates the warning in parser and throw the warning in tidb? |
Yes, you can remove the warning in parser. |
Index issue: #598
SQL example
Current implementation
AlterTableWithValidation
is ignored by TiDB.Expected implementation
Throw a warning only when the table is with validation.
The text was updated successfully, but these errors were encountered: