Merge pull request #45 from leungi/fix-get_partitions

fix: remove single quotes in SQL statement
ddotta · May 30, 2023 · 357c006 · 357c006
2 parents 4523002 + 4d611bc
commit 357c006
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,10 @@
+# parquetize (WIP)
+
+This release includes :
+
+- bugfix by @leungi: remove single quotes in SQL statement thatgenerates incorrect SQL syntax for connection of type Microsoft SQL Server #45
+
+
 # parquetize 0.5.6.1
 
 This release includes :

diff --git a/R/get_partitions.R b/R/get_partitions.R
@@ -29,5 +29,5 @@ get_partitions <- function(conn, table, column) {
     cli_abort("Be careful, the argument column must be filled in", class = "parquetize_missing_argument")
   }
 
-  DBI::dbGetQuery(conn, glue::glue("SELECT distinct({`column`}) FROM '{`table`}'", .con = conn))[,1]
+  DBI::dbGetQuery(conn, glue::glue("SELECT distinct({`column`}) FROM {`table`}", .con = conn))[,1]
 }