Revert "Fix docs for window functions"

This reverts commit e5771a1.

Author: junyangq

R/pkg/R/DataFrame.R

@@ -2483,8 +2483,7 @@ setMethod("unionAll",
 #'
 #' @param x a SparkDataFrame.
 #' @param ... additional SparkDataFrame(s).
-#' @param deparse.level currently not used (put here to match the signature of
-#'                      the base implementation).
+#' @param deparse.level dummy variable, currently not used.
 #' @return A SparkDataFrame containing the result of the union.
 #' @family SparkDataFrame functions
 #' @aliases rbind,SparkDataFrame-method
@@ -3213,7 +3212,7 @@ setMethod("histogram",
 #' @param x A SparkDataFrame
 #' @param url JDBC database url of the form `jdbc:subprotocol:subname`
 #' @param tableName The name of the table in the external database
-#' @param ... additional JDBC database connection properties.
+#' @param ... additional JDBC database connection propertie(s).
 #' @param mode One of 'append', 'overwrite', 'error', 'ignore' save mode (it is 'error' by default)
 #' @family SparkDataFrame functions
 #' @rdname write.jdbc

R/pkg/R/SQLContext.R

@@ -181,7 +181,7 @@ getDefaultSqlSource <- function() {
 #' @method createDataFrame default
 #' @note createDataFrame since 1.4.0
 # TODO(davies): support sampling and infer type from NA
-createDataFrame.default <- function(data, schema = NULL, samplingRatio = 1.0) {
+createDataFrame.default <- function(data, schema = NULL) {
   sparkSession <- getSparkSession()
   if (is.data.frame(data)) {
       # get the names of columns, they will be put into RDD

R/pkg/R/functions.R

@@ -1141,7 +1141,7 @@ setMethod("minute",
 #' @export
 #' @examples \dontrun{select(df, monotonically_increasing_id())}
 setMethod("monotonically_increasing_id",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "monotonically_increasing_id")
             column(jc)
@@ -1498,7 +1498,7 @@ setMethod("soundex",
 #' \dontrun{select(df, spark_partition_id())}
 #' @note spark_partition_id since 2.0.0
 setMethod("spark_partition_id",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "spark_partition_id")
             column(jc)
@@ -3128,7 +3128,7 @@ setMethod("ifelse",
 #' }
 #' @note cume_dist since 1.6.0
 setMethod("cume_dist",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "cume_dist")
             column(jc)
@@ -3152,7 +3152,7 @@ setMethod("cume_dist",
 #' @examples \dontrun{dense_rank()}
 #' @note dense_rank since 1.6.0
 setMethod("dense_rank",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "dense_rank")
             column(jc)
@@ -3267,7 +3267,7 @@ setMethod("ntile",
 #' @examples \dontrun{percent_rank()}
 #' @note percent_rank since 1.6.0
 setMethod("percent_rank",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "percent_rank")
             column(jc)
@@ -3325,7 +3325,7 @@ setMethod("rank",
 #' @examples \dontrun{row_number()}
 #' @note row_number since 1.6.0
 setMethod("row_number",
-          signature("missing"),
+          signature(),
           function() {
             jc <- callJStatic("org.apache.spark.sql.functions", "row_number")
             column(jc)

R/pkg/R/generics.R

@@ -920,10 +920,10 @@ setGeneric("crc32", function(x) { standardGeneric("crc32") })
 #' @export
 setGeneric("hash", function(x, ...) { standardGeneric("hash") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname cume_dist
 #' @export
-setGeneric("cume_dist", function(x = "missing") { standardGeneric("cume_dist") })
+setGeneric("cume_dist", function(...) { standardGeneric("cume_dist") })
 
 #' @rdname datediff
 #' @export
@@ -953,10 +953,10 @@ setGeneric("dayofyear", function(x) { standardGeneric("dayofyear") })
 #' @export
 setGeneric("decode", function(x, charset) { standardGeneric("decode") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname dense_rank
 #' @export
-setGeneric("dense_rank", function(x = "missing") { standardGeneric("dense_rank") })
+setGeneric("dense_rank", function(...) { standardGeneric("dense_rank") })
 
 #' @rdname encode
 #' @export
@@ -1070,11 +1070,11 @@ setGeneric("md5", function(x) { standardGeneric("md5") })
 #' @export
 setGeneric("minute", function(x) { standardGeneric("minute") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname monotonically_increasing_id
 #' @export
 setGeneric("monotonically_increasing_id",
-           function(x = "missing") { standardGeneric("monotonically_increasing_id") })
+           function(...) { standardGeneric("monotonically_increasing_id") })
 
 #' @rdname month
 #' @export
@@ -1108,10 +1108,10 @@ setGeneric("ntile", function(x) { standardGeneric("ntile") })
 #' @export
 setGeneric("n_distinct", function(x, ...) { standardGeneric("n_distinct") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname percent_rank
 #' @export
-setGeneric("percent_rank", function(x = "missing") { standardGeneric("percent_rank") })
+setGeneric("percent_rank", function(...) { standardGeneric("percent_rank") })
 
 #' @rdname pmod
 #' @export
@@ -1154,10 +1154,10 @@ setGeneric("reverse", function(x) { standardGeneric("reverse") })
 #' @export
 setGeneric("rint", function(x) { standardGeneric("rint") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname row_number
 #' @export
-setGeneric("row_number", function(x = "missing") { standardGeneric("row_number") })
+setGeneric("row_number", function(...) { standardGeneric("row_number") })
 
 #' @rdname rpad
 #' @export
@@ -1215,10 +1215,10 @@ setGeneric("sort_array", function(x, asc = TRUE) { standardGeneric("sort_array")
 #' @export
 setGeneric("soundex", function(x) { standardGeneric("soundex") })
 
-#' @param x empty. Should be used with no argument.
+#' @param ... empty. Use with no argument.
 #' @rdname spark_partition_id
 #' @export
-setGeneric("spark_partition_id", function(x = "missing") { standardGeneric("spark_partition_id") })
+setGeneric("spark_partition_id", function(...) { standardGeneric("spark_partition_id") })
 
 #' @rdname sd
 #' @export