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

    https://github.com/apache/spark/pull/21434#discussion_r191090863
  
    --- Diff: R/pkg/R/functions.R ---
    @@ -3048,6 +3048,26 @@ setMethod("array_position",
                 column(jc)
               })
     
    +#' @details
    +#' \code{array_repeat}: Creates an array containing the left argument 
repeated the number of times
    +#' given by the right argument.
    +#'
    +#' @param count Column or constant determining the number of repetitions.
    +#' @rdname column_collection_functions
    +#' @aliases array_repeat array_repeat,Column,numericOrColumn-method
    +#' @note array_repeat since 2.4.0
    +setMethod("array_repeat",
    +          signature(x = "Column", count = "numericOrColumn"),
    +          function(x, count) {
    +            if (class(count) == "Column") {
    +                count <- count@jc
    +            } else {
    +                count <- as.integer(count)
    --- End diff --
    
    indent is 2 space actually, could you update this and line L3063


---

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

Reply via email to