HyukjinKwon commented on a change in pull request #32797:
URL: https://github.com/apache/spark/pull/32797#discussion_r646294136



##########
File path: R/pkg/R/SQLContext.R
##########
@@ -378,6 +378,9 @@ setMethod("toDF", signature(x = "RDD"),
 #' ) is supported. For JSON (one record per file), set a named property 
\code{multiLine} to
 #' \code{TRUE}.
 #' It goes through the entire dataset once to determine the schema.
+#' You can find the JSON-specific options for reading JSON files in
+#' 
\href{https://spark.apache.org/docs/latest/sql-data-sources-json.html#data-source-option}{
+#' Data Source Option} in the version you use.

Review comment:
       can you move this to `@param ...`

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -406,6 +409,9 @@ read.json <- function(path, ...) {
 #' Create a SparkDataFrame from an ORC file.
 #'
 #' Loads an ORC file, returning the result as a SparkDataFrame.
+#' You can find the ORC-specific options for reading ORC files in

Review comment:
       ditto

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -427,6 +433,9 @@ read.orc <- function(path, ...) {
 #' Create a SparkDataFrame from a Parquet file.
 #'
 #' Loads a Parquet file, returning the result as a SparkDataFrame.
+#' You can find the Parquet-specific options for reading Parquet files in
+#' 
\href{https://spark.apache.org/docs/latest/sql-data-sources-parquet.html#data-source-option}{
+#' Data Source Option} in the version you use.

Review comment:
       ditto

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -450,6 +459,9 @@ read.parquet <- function(path, ...) {
 #' Loads text files and returns a SparkDataFrame whose schema starts with
 #' a string column named "value", and followed by partitioned columns if
 #' there are any. The text files must be encoded as UTF-8.
+#' You can find the text-specific options for reading text files in

Review comment:
       ditto




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to