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

    https://github.com/apache/spark/pull/22455#discussion_r220033577
  
    --- Diff: R/pkg/R/DataFrame.R ---
    @@ -244,11 +246,31 @@ setMethod("showDF",
     #' @note show(SparkDataFrame) since 1.4.0
     setMethod("show", "SparkDataFrame",
               function(object) {
    -            cols <- lapply(dtypes(object), function(l) {
    -              paste(l, collapse = ":")
    -            })
    -            s <- paste(cols, collapse = ", ")
    -            cat(paste(class(object), "[", s, "]\n", sep = ""))
    +            allConf <- sparkR.conf()
    +            if (!is.null(allConf[["spark.sql.repl.eagerEval.enabled"]]) &&
    --- End diff --
    
    you can also save off `foo <- 
allConf[["spark.sql.repl.eagerEval.enabled"]]` and reuse it, ditto for other 
cases


---

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

Reply via email to