[GitHub] [spark] bzhaoopenstack commented on a diff in pull request #37023: [TYPO-FIX] Make the 'sep' description better in read_csv of pyspark p…

[GitBox <gi...@apache.org>]

bzhaoopenstack commented on code in PR #37023:
URL: https://github.com/apache/spark/pull/37023#discussion_r910545293


##########
python/pyspark/pandas/namespace.py:
##########
@@ -238,7 +238,8 @@ def read_csv(
     path : str
         The path string storing the CSV file to be read.
     sep : str, default ‘,’
-        Delimiter to use. Must be a single character.
+        Delimiter to use. A character string(len(sep) >= 1) which is used to separate

Review Comment:
   > OK, fix the spacing at least, or just say "nonempty string"
   
   Yeah, I test on my local env and compare the output with pandas. ;-) . It works and depends on the spark native DataFrame ability.
   
   And I agree with you about the simple description looks better, thanks.



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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org