raulcd commented on code in PR #46261:
URL: https://github.com/apache/arrow/pull/46261#discussion_r2123780318
##########
r/src/compute.cpp:
##########
@@ -618,6 +622,13 @@ SEXP compute__CallFunction(std::string func_name,
cpp11::list args, cpp11::list
// [[arrow::export]]
std::vector<std::string> compute__GetFunctionNames() {
+#if ARROW_VERSION_MAJOR >= 21
+ auto compute_init_status_ = arrow::compute::Initialize();
Review Comment:
This is a good point. I found this point to be necessary for our R builds
and tests but the registration on the R bindings might be better suited
somewhere else. @thisisnic @jonkeane @nealrichardson @paleolimbot any
suggestion around the best place to register the compute functions on the R
bindings?
--
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
