Skip to content

Addition of USE EXEC OPENQUERY syntax - #8046

Merged
Martin Aeschlimann (aeschli) merged 1 commit into
microsoft:masterfrom
gontadu:patch-1
Jun 27, 2016
Merged

Addition of USE EXEC OPENQUERY syntax#8046
Martin Aeschlimann (aeschli) merged 1 commit into
microsoft:masterfrom
gontadu:patch-1

Conversation

@gontadu

Copy link
Copy Markdown
Contributor

USE is used to select a database in SQL Server when a database is connected to.
EXEC and OPENQUERY are both used to get data on other servers.

I have grouped this with the VALUES keyword as EXEC/OPENQUERY are dealing with similar data.
I have also added USE into this dictionart as I felt it is cleaner than adding into dictionary keyword.other.DML.sql but let me know if you disagree.

USE is used to select a database in SQL Server when a database is connected to.
EXEC and OPENQUERY are both used to get data on other servers.

I have grouped this with the VALUES keyword as EXEC/OPENQUERY are dealing with similar data.
I have also added USE into this dictionart as I felt it is cleaner than adding into dictionary keyword.other.DML.sql but let me know if you disagree.
@mention-bot

Copy link
Copy Markdown

By analyzing the blame information on this pull request, we identified Erich Gamma (@egamma) to be a potential reviewer

@msftclas

Hi Eshwar Andhavarapu (@gontadu), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@aeschli

Copy link
Copy Markdown
Contributor

Eshwar Andhavarapu (@gontadu) Thanks for the pull request, can you also make this request against https://github.com/textmate/sql.tmbundle/ ?
Thanks a lot!

@aeschli
Martin Aeschlimann (aeschli) merged commit fef7e1c into microsoft:master Jun 27, 2016
@gontadu

Eshwar Andhavarapu (gontadu) commented Jun 28, 2016

Copy link
Copy Markdown
Contributor Author

Ok will do now!

DONE: textmate/sql.tmbundle#15

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants