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]