Skip to content
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

Closed
tangenta opened this issue Oct 31, 2019 · 7 comments · Fixed by pingcap/tidb#16089 or #795
Closed

Fix TiDB behavior: with validation #606

tangenta opened this issue Oct 31, 2019 · 7 comments · Fixed by pingcap/tidb#16089 or #795
Labels
good first issue Good for newcomers

Comments

@tangenta
Copy link
Contributor

Index issue: #598

SQL example

ALTER TABLE t WITH VALIDATION;

Current implementation

AlterTableWithValidation is ignored by TiDB.

Expected implementation

Throw a warning only when the table is with validation.

@yuanjize
Copy link

Should these problem about incompatible statements be fixed in repo parser or repo tidb?

@tangenta
Copy link
Contributor Author

@yuanjize We need to update the code in TiDB repo.

@tangenta
Copy link
Contributor Author

@yuanjize If you have any questions, feel free to ask here or contact me on slack :).

Slack: channel #sig-parser in the tidbcommunity slack workspace.

@cncal
Copy link
Contributor

cncal commented Mar 28, 2020

@tangenta , I'm working on this issue as a new contributor. A warning is threw when I execute ALTER TABLE t WITH VALIDATION; and the warning is as follows:

1064 You have an error in your SQL syntax; check the manual that corresponds to your TiDB version for the right syntax to use line 1 column 30 near ""The WITH/WITHOUT VALIDATION clause is parsed but ignored by all storage engines.

Has this issue been solved already?

@tangenta
Copy link
Contributor Author

@cncal This issue has not been fixed yet. Your contribution is welcomed :)

@cncal
Copy link
Contributor

cncal commented Mar 30, 2020

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?

@tangenta
Copy link
Contributor Author

Yes, you can remove the warning in parser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
3 participants