itholic commented on code in PR #36041:
URL: https://github.com/apache/spark/pull/36041#discussion_r843472975


##########
python/pyspark/pandas/series.py:
##########
@@ -2154,6 +2159,8 @@ def clip(self, lower: Union[float, int] = None, upper: 
Union[float, int] = None)
             Minimum threshold value. All values below this threshold will be 
set to it.
         upper : float or int, default None
             Maximum threshold value. All values above this threshold will be 
set to it.
+        inplace : bool, default False
+             if True, perform operation in-place

Review Comment:
   Sure. Let's keep it as is for now.
   
   Although I suggested to add them just in case, maybe using `*args` and 
`**kwargs` here and other APIs is fairly uncommon.



-- 
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: [email protected]

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