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

    https://github.com/apache/spark/pull/21482#discussion_r192574568
  
    --- Diff: R/pkg/R/functions.R ---
    @@ -907,6 +907,30 @@ setMethod("initcap",
                 column(jc)
               })
     
    +#' @details
    +#' \code{isinf}: Returns true if the column is Infinity.
    +#' @rdname column_nonaggregate_functions
    +#' @aliases isnan isnan,Column-method
    +#' @note isinf since 2.4.0
    +setMethod("isinf",
    --- End diff --
    
    We don't need to have duplicate method definition like this. Maybe we can 
follow `isNaN`'s approach if we really need to have both `isinf` and `isInf`.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to