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

    https://github.com/apache/spark/pull/21142#discussion_r183952708
  
    --- Diff: R/pkg/R/functions.R ---
    @@ -206,6 +206,8 @@ NULL
     #' df <- createDataFrame(cbind(model = rownames(mtcars), mtcars))
     #' tmp <- mutate(df, v1 = create_array(df$mpg, df$cyl, df$hp))
     #' head(select(tmp, array_contains(tmp$v1, 21), size(tmp$v1)))
    +#' head(select(tmp, array_max(tmp$v1)))
    +#' head(select(tmp, array_min(tmp$v1)))
    --- End diff --
    
    let's combine these into one line? I think multiple examples in the pattern 
of "head(select(...))" isn't super useful to be honest


---

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

Reply via email to