Making querySql() less inefficient for large queries.

Author: Admin_mschuemi

NEWS.md

@@ -11,6 +11,8 @@ Changes:
 
 4. Updating IRIS JDBC driver.
 
+5. Making `querySql()` less inefficient for large queries.
+
 Bugfixes:
 
 1. Converting double quotes to spaces when upload to DataBricks to avoid errors.

R/LowLevelFunctions.R

@@ -181,12 +181,13 @@ getAllBatches <- function(batchedQuery) {
   if (any(columnTypes == 5)) {
     validateInt64Query()
   }
-  columns <- data.frame()
+  data <- list()
   while (!rJava::.jcall(batchedQuery, "Z", "isDone")) {
     rJava::.jcall(batchedQuery, "V", "fetchBatch")
     batch <- parseJdbcColumnData(batchedQuery,
                                  columnTypes = columnTypes)
-    columns <- rbind(columns, batch)
+    data[[length(data) + 1]] <- batch
   }
-  return(columns)
+  data <- bind_rows(data)
+  return(data)
 }