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

    https://github.com/apache/spark/pull/22455#discussion_r219689697
  
    --- Diff: R/pkg/R/DataFrame.R ---
    @@ -244,11 +246,25 @@ 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 = ""))
    +            if (identical(sparkR.conf("spark.sql.repl.eagerEval.enabled", 
"false")[[1]], "true")) {
    +              argsList <- list()
    +              argsList$x <- object
    +              numRows <- 
as.numeric(sparkR.conf("spark.sql.repl.eagerEval.maxNumRows", "0")[[1]])
    +              if (numRows > 0) {
    +                argsList$numRows <- numRows
    +              }
    +              truncate <- 
as.numeric(sparkR.conf("spark.sql.repl.eagerEval.truncate", "0")[[1]])
    --- End diff --
    
    have you tested with the case when only `spark.sql.repl.eagerEval.truncate` 
is set, and `spark.sql.repl.eagerEval.maxNumRows` is not?


---

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

Reply via email to