Github user felixcheung commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22954#discussion_r231402726
  
    --- Diff: R/pkg/R/SQLContext.R ---
    @@ -147,6 +147,30 @@ getDefaultSqlSource <- function() {
       l[["spark.sql.sources.default"]]
     }
     
    +writeToTempFileInArrow <- function(rdf, numPartitions) {
    +  stopifnot(require("arrow", quietly = TRUE))
    +  stopifnot(require("withr", quietly = TRUE))
    +  numPartitions <- if (!is.null(numPartitions)) {
    +    numToInt(numPartitions)
    +  } else {
    +    1
    +  }
    +  fileName <- tempfile()
    --- End diff --
    
    might need to give it a dir prefix to use - the tempfile default is not 
CRAN compliant and possibly some ACL issue


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to