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



##########
File path: R/pkg/R/DataFrame.R
##########
@@ -920,6 +924,10 @@ setMethod("write.json",
 #' @param mode one of 'append', 'overwrite', 'error', 'errorifexists', 'ignore'
 #'             save mode (it is 'error' by default)
 #' @param ... additional argument(s) passed to the method.
+#'            You can find the ORC-specific options for writing ORC files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```

##########
File path: R/pkg/R/DataFrame.R
##########
@@ -951,6 +959,10 @@ setMethod("write.orc",
 #' @param mode one of 'append', 'overwrite', 'error', 'errorifexists', 'ignore'
 #'             save mode (it is 'error' by default)
 #' @param ... additional argument(s) passed to the method.
+#'            You can find the Parquet-specific options for writing Parquet 
files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```

##########
File path: R/pkg/R/DataFrame.R
##########
@@ -983,6 +995,10 @@ setMethod("write.parquet",
 #' @param mode one of 'append', 'overwrite', 'error', 'errorifexists', 'ignore'
 #'             save mode (it is 'error' by default)
 #' @param ... additional argument(s) passed to the method.
+#'            You can find the text-specific options for writing text files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```

##########
File path: R/pkg/R/DataFrame.R
##########
@@ -3731,6 +3747,9 @@ setMethod("histogram",
 #'
 #' Save the content of the SparkDataFrame to an external database table via 
JDBC. Additional JDBC
 #' database connection properties can be set (...)
+#' You can find the JDBC-specific option and parameter documentation for 
writing tables via JDBC in
+#' 
\href{https://spark.apache.org/docs/latest/sql-data-sources-jdbc.html#data-source-option}{

Review comment:
       ```suggestion
   #' 
\url{https://spark.apache.org/docs/latest/sql-data-sources-jdbc.html#data-source-option}{
   ```

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -381,6 +381,10 @@ setMethod("toDF", signature(x = "RDD"),
 #'
 #' @param path Path of file to read. A vector of multiple paths is allowed.
 #' @param ... additional external data source specific named properties.
+#'            You can find the JSON-specific options for reading JSON files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -409,6 +413,10 @@ read.json <- function(path, ...) {
 #'
 #' @param path Path of file to read.
 #' @param ... additional external data source specific named properties.
+#'            You can find the ORC-specific options for reading ORC files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```

##########
File path: R/pkg/R/SQLContext.R
##########
@@ -430,6 +438,10 @@ read.orc <- function(path, ...) {
 #'
 #' @param path path of file to read. A vector of multiple paths is allowed.
 #' @param ... additional data source specific named properties.
+#'            You can find the Parquet-specific options for reading Parquet 
files in
+#'            \href{

Review comment:
       ```suggestion
   #'            \url{
   ```




-- 
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