HyukjinKwon commented on a change in pull request #32723: URL: https://github.com/apache/spark/pull/32723#discussion_r643577724
########## File path: sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ########## @@ -301,23 +306,22 @@ class DataFrameReader private[sql](sparkSession: SparkSession) extends Logging { * Don't create too many partitions in parallel on a large cluster; otherwise Spark might crash * your external database systems. * - * @param url JDBC database url of the form `jdbc:subprotocol:subname`. + * You can find the JDBC-specific options for reading table via JDBC in Review comment: Can we change: "JDBC-specific options for reading table" -> "JDBC-specific option and parameter documentation for reading tables"? ########## File path: sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ########## @@ -282,6 +282,10 @@ class DataFrameReader private[sql](sparkSession: SparkSession) extends Logging { * Construct a `DataFrame` representing the database table accessible via JDBC URL * url named table and connection properties. * + * You can find the JDBC-specific options for reading table via JDBC in Review comment: reading a table or reading tables -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org