raulcd commented on code in PR #46261:
URL: https://github.com/apache/arrow/pull/46261#discussion_r2102544207


##########
c_glib/arrow-glib/compute.cpp:
##########
@@ -37,6 +37,9 @@
 #include <arrow/acero/exec_plan.h>
 #include <arrow/acero/options.h>
 
+// Initialize the compute library and register compute kernels.
+auto compute_init_status_ = arrow::compute::Initialize();

Review Comment:
   @kou I don't know much about the CGLib implementation. What would be the 
best way to check for the status here? Should I add a function like:
   ```
   gboolean
   garrow_compute_is_initialized(void)
   {
     return compute_init_status_.ok();
   }
   ```
   but how would we check this? Should I validate the initialization on the 
different methods that require compute?



-- 
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

Reply via email to