pitrou commented on a change in pull request #8271:
URL: https://github.com/apache/arrow/pull/8271#discussion_r500268258



##########
File path: python/pyarrow/compute.py
##########
@@ -253,6 +255,68 @@ def match_substring(array, pattern):
                          MatchSubstringOptions(pattern))
 
 
+def split_pattern(array, pattern, max_splits=-1, reverse=False):

Review comment:
       It's true that the docstring is a bit suboptimal still. Hopefully we can 
improve that in the future:
   ```
   >>> pc.min_max?
   Signature: pc.min_max(arg, *, options=None, memory_pool=None, **kwargs)
   Docstring:
   Call compute function 'min_max' with the given argument.
   
   Parameters
   ----------
   arg : Array-like or scalar-like
       Argument to compute function
   memory_pool : pyarrow.MemoryPool, optional
       If not passed, will allocate memory from the default memory pool.
   options : pyarrow.compute.MinMaxOptions, optional
       Parameters altering compute function semantics
   **kwargs: optional
       Parameters for MinMaxOptions constructor.  Either `options`
       or `**kwargs` can be passed, but not both at the same time.
   ```




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


Reply via email to