maartenbreddels commented on a change in pull request #8459:
URL: https://github.com/apache/arrow/pull/8459#discussion_r506696199
##########
File path: cpp/CMakeLists.txt
##########
@@ -735,6 +735,7 @@ endif()
if(ARROW_WITH_UTF8PROC)
list(APPEND ARROW_LINK_LIBS utf8proc::utf8proc)
+ list(APPEND ARROW_LINK_LIBS re2)
Review comment:
I guess that also means it is better we have kernels that refer to RE2
being used as I did in #8468 (replace_substring and replace_substring_re2)
instead of having a kernel that will bail out at runtime because re2 was not
enabled.
----------------------------------------------------------------
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]
