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.

Sign up for GitHub

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

Jump to bottom

Allow "clustering" as a keyspace name #259

Closed
m-iancu opened this issue Apr 28, 2018 · 1 comment
Closed

Allow "clustering" as a keyspace name #259

m-iancu opened this issue Apr 28, 2018 · 1 comment
Assignees
@m-iancu

Comments

@m-iancu
Copy link
Contributor

m-iancu commented Apr 28, 2018

It is a legal name in Cassandra but in YugaByte we currently do not allow it:

cqlsh> create keyspace clustering;
SyntaxException: SQL error: Invalid SQL Statement. syntax error, unexpected CLUSTERING, expecting AUTHORIZATION
create keyspace clustering;
                ^^^^^^^^^^
 (error -11)
@m-iancu m-iancu self-assigned this Apr 28, 2018
yugabyte-ci pushed a commit that referenced this issue May 2, 2018
@m-iancu
#259 Fix inconsistencies in YCQL keyword list
477c3e3 
Summary:
Make sure the declared type of YCQL keywords (reserved, unreserved, col_name, etc.) matches
with their use in the parser.

Test Plan: ql-parser-test

Reviewers: neil, robert

Reviewed By: robert

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D4737
@m-iancu
Copy link
Contributor Author

m-iancu commented May 2, 2018

Closed by 477c3e3

@m-iancu m-iancu closed this as completed May 2, 2018
jasonyb pushed a commit that referenced this issue Jun 11, 2024
@ImTheKai
Merge pull request #259 from puneet0191/PMM-9951-4
55ba2de 
PMM-9951 moving integration tests to push
devansh-ism pushed a commit to devansh-ism/yugabyte-db that referenced this issue Jul 17, 2024
@mattdeluco
Follow up to yugabyte#216 (fixed in 007ee2f). Fixes additional test f…
3064044 
…ailures (yugabyte#259)

where the current username must be quoted as an identifier.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-iancu m-iancu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@m-iancu