fsaintjacques edited a comment on pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#issuecomment-632825223


   @nealrichardson the next step is probably to break `arrowExport.{h,cpp}` 
into one per file cpp file, e.g.
   
   ```
   src/buffer.cpp
   src/buffer.h
   src/buffer_export.cpp
   ...
   ```
   
   This will make the export file much smaller and require less headers to 
read, on top of allowing this to be parallelized (as opposed to a single thread 
for the huge file as it is). I suspect we can shave probably another 33%, the 
final link will still be a bottleneck. I'm not familiar with rcpp and it's 
restrictions/requirements, @romainfrancois might say if this is never going to 
work.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to