[GitHub] [arrow] cyb70289 commented on a change in pull request #10448: ARROW-12949: [C++] Add starts_with and ends_with

[GitBox <gi...@apache.org>]

cyb70289 commented on a change in pull request #10448:
URL: https://github.com/apache/arrow/pull/10448#discussion_r645305540



##########
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##########
@@ -484,6 +484,42 @@ struct PlainSubstringMatcher {
   bool Match(util::string_view current) const { return Find(current) >= 0; }
 };
 
+struct PlainStartsWithMatcher {
+  const MatchSubstringOptions& options_;
+
+  explicit PlainStartsWithMatcher(const MatchSubstringOptions& options)
+      : options_(options) {}
+
+  static Result<std::unique_ptr<PlainStartsWithMatcher>> Make(
+      const MatchSubstringOptions& options) {
+    // Should be handled by partial template specialization below

Review comment:
       Not sure what this comment means. Looks there's no template around.




-- 
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:
users@infra.apache.org