jorisvandenbossche commented on pull request #8459:
URL: https://github.com/apache/arrow/pull/8459#issuecomment-823544553


   >  Do you think "extract_regex" is the right term for this function? 
Something else?
   
   In pandas, this is called "extract" 
(https://pandas.pydata.org/docs/reference/api/pandas.Series.str.extract.html), 
so from that point of view this is a good name. 
   In `re`, I think one would use `re.search`, but that's basically both used 
for "match" as for "extract" purposes, so I think "extract" is a better name 
for the kernel in this PR. 
   
   The `_regex` might be a bit superfluous, as there is no variant without 
regex (but on the other hand it explicitly signals it is using regular 
expressions).


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