Ziy1-Tan commented on PR #46813: URL: https://github.com/apache/arrow/pull/46813#issuecomment-2976787230
> Thanks for tackling the work. Part of the current problem on CI is with calling the file `*_internal.h` . The header file won't be installed and this is used on some of the bindings which requires it. We can move the namespace to `arrow::internal` but keep installing the header file, probably naming the files `string_util.cc` and `string_util.h`? Thank you for your review. Let me take a look. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
