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

sql: unreserve the STORED keyword #24554

Merged
merged 1 commit into from
Apr 17, 2018
Merged

Conversation

jordanlewis
Copy link
Member

@jordanlewis jordanlewis commented Apr 6, 2018

Release note (sql change): the word stored can again be used to name
databases, tables or columns without requiring clients to quote the
identifier.

@jordanlewis jordanlewis requested review from bdarnell, knz and a team April 6, 2018 16:09
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@petermattis
Copy link
Collaborator

Are we sure LATERAL doesn't need to be a reserved word? It isn't even used by the grammar right now, but it will be when we support LATERAL JOINS (possibly happening in 2.1).


Review status: 0 of 2 files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@bdarnell
Copy link
Contributor

bdarnell commented Apr 6, 2018

Yeah, LATERAL is a reserved word in postgres (according to https://www.postgresql.org/docs/current/static/sql-keywords-appendix.html. I was working from a less authoritative source when I said it wasn't reserved).

Release note (sql change): the word `stored` can again be used to name
databases, tables or columns without requiring clients to quote the
identifier.
@knz knz changed the title sql: unreserve LATERAL and STORED keywords sql: unreserve the STORED keyword Apr 17, 2018
@knz
Copy link
Contributor

knz commented Apr 17, 2018

amended to only unreserve STORED.

bors r+

craig bot pushed a commit that referenced this pull request Apr 17, 2018
24554: sql: unreserve the STORED keyword r=knz a=jordanlewis

Release note (sql change): the word `stored` can again be used to name
databases, tables or columns without requiring clients to quote the
identifier.

Co-authored-by: Jordan Lewis <[email protected]>
@craig
Copy link
Contributor

craig bot commented Apr 17, 2018

Build succeeded

@craig craig bot merged commit 3ac8053 into cockroachdb:master Apr 17, 2018
craig bot pushed a commit that referenced this pull request Apr 17, 2018
24864: cherry-pick 2.0: sql: unreserve the STORED keyword r=knz a=knz

Picks #24554.

cc @cockroachdb/release 

Co-authored-by: Jordan Lewis <[email protected]>
@jordanlewis jordanlewis deleted the unreserve branch May 21, 2018 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants