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



##########
File path: python/pyspark/sql/readwriter.py
##########
@@ -627,8 +627,6 @@ def jdbc(self, url, table, column=None, lowerBound=None, 
upperBound=None, numPar
 
         Parameters
         ----------
-        url : str
-            a JDBC URL of the form ``jdbc:subprotocol:subname``
         table : str
             the name of the table
         column : str, optional

Review comment:
       I think we can remove `lowerBound`, `upperBound`, and `numPartitions`.
   And, fix the description of `column` to something like:
   
   Alias of `partitionColumn` option. Refer to `partitionColumn` in `Data 
Source Option <...>`_ in the version you use.




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

Reply via email to