-
Notifications
You must be signed in to change notification settings - Fork 42
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
Table name which is also a "reserved word" fails to parse #108
Comments
@nurih I will fix it in next release, it should not recognise it as INDEX token in this statement. Thanks for opening the issue! |
@nurih can you clarify for me about 'col1:int' syntax, what DB support it? did not saw before that column name & type separated by ':'. |
@nurih I added the test for case """create table index (col1 int); |
|
Very happy for this quick fix! Thank you! |
Consider:
The first 2 lines understand a table named "index" is intended.
The third line classifies the word "index" as the token 'INDEX' probably here.
Though not a good or recommended practice, table names are allowed to be arbitrary strings, even reserved words.
If strict adherence to quoted/delimited usage of key-word-as-identifier DDL is desired, then the first line (bare "index" ) should have failed.
If relaxed adherence is intended, then all 3 should have succeeded.
The text was updated successfully, but these errors were encountered: