edponce commented on a change in pull request #11233:
URL: https://github.com/apache/arrow/pull/11233#discussion_r728485872
##########
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##########
@@ -2902,9 +2929,9 @@ struct ExtractRegexData {
// No input type specified => propagate shape
return args[0];
}
- // Input type is either String or LargeString and is also the type of each
+ // Input type is [Large]Binary or [Large]String and is also the type of
each
Review comment:
This is more than a "nit". Need to update comments and docs to be
consistent with kernels type support.
--
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]
