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

CREATE TABLE accepts empty table/column name #6653

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

CREATE TABLE accepts empty table/column name #6653

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2018-10-22 17:25:05 +0200
From: @kutsurak
To: SQL devs <>
Version: 11.31.7 (Aug2018)

Last updated: 2019-01-14 17:29:06 +0100

Comment 26647

Date: 2018-10-22 17:25:05 +0200
From: @kutsurak

The statement:

CREATE TABLE tbl ("" INT);

succeeds, but the SQL standard does not allow this since identifiers cannot be empty. The column definition comes down to a nonterminal in the grammar, which at minimum is four (4) double quotes in a row:

::=

::=
...

::=

|

::=
""!! two consecutive double quote characters

Note that the statement

CREATE TABLE "" (i int);

works as well, where it probably should not for the same reason as above.

Comment 26648

Date: 2018-10-22 17:42:30 +0200
From: MonetDB Mercurial Repository <>

Changeset 86984307106e made by Panagiotis Koutsourakis kutsurak@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=86984307106e

Changeset description:

Add test for bug #6653

Comment 26653

Date: 2018-10-24 17:07:14 +0200
From: MonetDB Mercurial Repository <>

Changeset 6f22c3a68031 made by Pedro Ferreira pedro.ferreira@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=6f22c3a68031

Changeset description:

Fixes for bug #6653 (i.e. disallow any empty identifier).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant