Support non-temporary writes for databricks

Fixes #1418
tidyverse · Jan 11, 2024 · 3563fa7 · 3563fa7
1 parent 781d630
commit 3563fa7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,7 @@
 # dbplyr (development version)
 
+* The databricks backend now supports creating non-temporary tables too (#1418).
+
 * Namespaced dplyr calls now error if the function doesn't exist, or 
   a translation is not available (#1426).
 

diff --git a/R/backend-spark-sql.R b/R/backend-spark-sql.R
@@ -126,12 +126,12 @@ simulate_spark_sql <- function() simulate_dbi("Spark SQL")
                                    analyze = TRUE,
                                    in_transaction = FALSE) {
 
-  if (!temporary) {
-    cli::cli_abort("Spark SQL only support temporary tables")
+  if (temporary) {
+    sql <- sql_values_subquery(con, values, types = types, lvl = 1)
+    db_compute(con, table, sql, overwrite = overwrite)
+  } else {
+    NextMethod()
   }
-
-  sql <- sql_values_subquery(con, values, types = types, lvl = 1)
-  db_compute(con, table, sql, overwrite = overwrite)
 }
 
 #' @export