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

sql: make it possible to refer to dbs/tables/indices by number #12715

Closed
knz opened this issue Jan 5, 2017 · 1 comment
Closed

sql: make it possible to refer to dbs/tables/indices by number #12715

knz opened this issue Jan 5, 2017 · 1 comment
Assignees
@knz
Labels
A-sql-semantics C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Milestone
Later

Comments

@knz
Copy link
Contributor

knz commented Jan 5, 2017

Like we already have ordinal references for data sources we should have ordinal references to dbs/tables/indices.

Like e.g. SELECT * FROM !1231 to select from table 1231. or SELECT !1231 FROM foo for column 1231 or SELECT * FROM !1231.foo for a db ref, or SELECT * FROM foo@!1231 for an index ref.

Will help with IR work and also will simplify the code produced for #12682.

cc @dt
@knz knz added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-sql-semantics labels Jan 5, 2017
@knz knz self-assigned this Jan 5, 2017
@knz
Copy link
Contributor Author

knz commented Jan 5, 2017

This may help make view descriptors stable across table/index/column renames.

@cuongdo cuongdo added this to the Later milestone Feb 22, 2017
This was referenced Feb 23, 2017
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@knz knz

Labels
A-sql-semantics C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Projects
None yet
Milestone
Later
Development

No branches or pull requests
2 participants
@knz @cuongdo