Skip to content

Commit

Permalink
Fix check for managed instance and update version number (#69)
Browse files Browse the repository at this point in the history
  • Loading branch information
JonathanZhu11 authored Jun 15, 2020
1 parent 57c0009 commit 50e09d9
Show file tree
Hide file tree
Showing 16 changed files with 74 additions and 29 deletions.
4 changes: 2 additions & 2 deletions R/DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Package: sqlmlutils
Type: Package
Title: Wraps R code into executable SQL Server stored procedures
Version: 0.7.1
Version: 0.7.2
Author: Microsoft Corporation
Maintainer: Microsoft Corporation <msrpack@microsoft.com>
Depends:
Expand All @@ -13,7 +13,7 @@ Description: sqlmlutils is a package designed to help users interact with SQL Se
creating and running stored procedures, and managing packages on the database.
License: MIT + file LICENSE
Copyright: Copyright 2016 Microsoft Corporation
RoxygenNote: 6.1.1
RoxygenNote: 7.1.0
Encoding: UTF-8
Suggests: testthat (>= 2.0.0),
roxygen2
2 changes: 1 addition & 1 deletion R/R/sqlPackage.R
Original file line number Diff line number Diff line change
Expand Up @@ -785,7 +785,7 @@ sqlPackageManagementVersion <- function(connectionString)
stop(sprintf("Failed to get SQL version using connection string '%s'", connectionString ), call. = FALSE)
}

if(serverProperties[["edition"]] == "sql azure" && serverProperties[["engineEdition"]]==5)
if(serverProperties[["edition"]] == "sql azure" && (serverProperties[["engineEdition"]]==5 || serverProperties[["engineEdition"]]==8))
{
# sql azure
pmversion <- append(list(serverType = "azure"), tail(serverProperties, -2))
Expand Down
4 changes: 2 additions & 2 deletions R/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ sqlmlutils is an R package to help execute R code on a SQL database (SQL Server
From command prompt, run
```
R.exe -e "install.packages('RODBCext', repos='https://mran.microsoft.com/snapshot/2019-02-01/')"
R.exe CMD INSTALL dist/sqlmlutils_0.7.1.zip
R.exe CMD INSTALL dist/sqlmlutils_0.7.2.zip
```
OR
To build a new package file and install, run
Expand All @@ -19,7 +19,7 @@ To build a new package file and install, run
### Linux
```
R.exe -e "install.packages('RODBCext', repos='https://mran.microsoft.com/snapshot/2019-02-01/')"
R.exe CMD INSTALL dist/sqlmlutils_0.7.1.tar.gz
R.exe CMD INSTALL dist/sqlmlutils_0.7.2.tar.gz
```

# Getting started
Expand Down
2 changes: 0 additions & 2 deletions R/buildandinstall.cmd
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,5 @@ pushd .
cd ..
R -e "install.packages('RODBCext', repos='https://mran.microsoft.com/snapshot/2019-02-01/')"
R CMD INSTALL --build R
R CMD INSTALL --build R sqlmlutils.tar.gz
mv sqlmlutils_*.zip R/dist
mv sqlmlutils_*.tar.gz R/dist
popd
Binary file removed R/dist/sqlmlutils_0.7.1.tar.gz
Binary file not shown.
Binary file removed R/dist/sqlmlutils_0.7.1.zip
Binary file not shown.
Binary file added R/dist/sqlmlutils_0.7.2.tar.gz
Binary file not shown.
Binary file added R/dist/sqlmlutils_0.7.2.zip
Binary file not shown.
9 changes: 7 additions & 2 deletions R/man/connectionInfo.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

22 changes: 17 additions & 5 deletions R/man/createSprocFromFunction.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 7 additions & 2 deletions R/man/executeFunctionInSQL.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 6 additions & 2 deletions R/man/executeScriptInSQL.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions R/man/sql_install.packages.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions R/man/sql_installed.packages.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions R/man/sql_remove.packages.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ Windows:
From command prompt, run
```
R.exe -e "install.packages('RODBCext', repos='https://mran.microsoft.com/snapshot/2019-02-01/')"
R.exe CMD INSTALL dist/sqlmlutils_0.7.1.zip
R.exe CMD INSTALL dist/sqlmlutils_0.7.2.zip
```
OR
To build a new package file and install, run
Expand All @@ -40,7 +40,7 @@ To build a new package file and install, run
Linux
```
R.exe -e "install.packages('RODBCext', repos='https://mran.microsoft.com/snapshot/2019-02-01/')"
R.exe CMD INSTALL dist/sqlmlutils_0.7.1.tar.gz
R.exe CMD INSTALL dist/sqlmlutils_0.7.2.tar.gz
```

# Details
Expand Down

0 comments on commit 50e09d9

Please sign in to comment.