pitrou commented on a change in pull request #12698:
URL: https://github.com/apache/arrow/pull/12698#discussion_r836534470
##########
File path: python/pyarrow/_compute.pyx
##########
@@ -2127,21 +2138,76 @@ cdef class Expression(_Weakrefable):
return Expression._call("divide_checked", [self, other])
def is_valid(self):
- """Checks whether the expression is not-null (valid)"""
+ """
+ Check whether the expression is not-null (valid).
+
+ This creates a new expression equivalent to calling the
+ `is_valid` compute function on this expression.
+
+ Returns
+ -------
+ is_valid : Expression
Review comment:
No, it seems to pass actually. Given that other docstrings used this
convention, I just thought it was preferred.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
