Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Addition of T-SQL syntax: USE, GO, INTO, EXEC, OPENQUERY #36

Merged
merged 2 commits into from Jul 15, 2016
Merged

Addition of T-SQL syntax: USE, GO, INTO, EXEC, OPENQUERY #36

merged 2 commits into from Jul 15, 2016

Conversation

gontadu
Copy link
Contributor

@gontadu gontadu commented Jun 23, 2016

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.
@50Wliu
Copy link
Contributor

50Wliu commented Jun 24, 2016

I have no idea what DML even stands for but if you think USE fits in with keyword.other.DML.sql more than keyword.other.DML.II.sql then feel free to move it.

@gontadu
Copy link
Contributor Author

gontadu commented Jun 25, 2016

From my understanding DML is the superset of languages like SQL (languages that sound like SELECT... FROM, etc.) So for now until someone more educated in this field corrects us, let's leave in DML.II

@gontadu
Copy link
Contributor Author

gontadu commented Jul 6, 2016

Any update here?

@gontadu gontadu changed the title Addition of USE EXEC OPENQUERY syntax Addition of USE, GO, INTO, EXEC, OPENQUERY syntax Jul 12, 2016
@gontadu gontadu changed the title Addition of USE, GO, INTO, EXEC, OPENQUERY syntax Addition of T-SQL syntax: USE, GO, INTO, EXEC, OPENQUERY Jul 12, 2016
@50Wliu
Copy link
Contributor

50Wliu commented Jul 15, 2016

Sorry, was on vacation.

So for now until someone more educated in this field corrects us, let's leave in DML.II

Sounds like a good compromise.

@50Wliu 50Wliu merged commit 70c89c7 into atom:master Jul 15, 2016
@gontadu
Copy link
Contributor Author

gontadu commented Jul 16, 2016

Perfect! Happy with this! hope you had a great vacation!

@gontadu gontadu deleted the patch-2 branch July 22, 2016 20:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants