[GitHub] [arrow] bkietz commented on pull request #8371: ARROW-4960: [R] Build r-arrow conda package in crossbow

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

bkietz commented on pull request #8371:
URL: https://github.com/apache/arrow/pull/8371#issuecomment-707191975


   @xhochy @nealrichardson I'm probably misunderstanding something but I'm somewhat surprised we need the symbols to be visible; `arrowExports.cpp` includes an explicit lookup table:
   
   ```c++
   static const R_CallMethodDef CallEntries[] = {
   // ...
   		{ "_arrow_all_record_batches", (DL_FUNC) &_arrow_all_record_batches, 1}, 
   		{ "_arrow_Table__from_record_batches", (DL_FUNC) &_arrow_Table__from_record_batches, 2}, 
   		{ "_arrow_Table__from_dots", (DL_FUNC) &_arrow_Table__from_dots, 2}, 
   		{ "_arrow_GetCpuThreadPoolCapacity", (DL_FUNC) &_arrow_GetCpuThreadPoolCapacity, 0}, 
   		{ "_arrow_SetCpuThreadPoolCapacity", (DL_FUNC) &_arrow_SetCpuThreadPoolCapacity, 1}, 
   		{NULL, NULL, 0}
   };
   
   extern "C" void R_init_arrow(DllInfo* dll){
     R_registerRoutines(dll, NULL, CallEntries, NULL, NULL);
     R_useDynamicSymbols(dll, FALSE);
   }
   ```


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