nealrichardson opened a new pull request #7245: URL: https://github.com/apache/arrow/pull/7245
Rather than `#including` everything in one header file and `#including` that in every .cpp file, this breaks out those includes to each .cpp file. There's also a more limited `arrow_exports.h` now that is included in the `arrowExports.cpp` that gets generated. Installation/compilation time for the R bindings on my laptop (single-threaded) dropped from 130s to 90s (roughly 1/3 less, 40s total) as a result, so it seems like we were wasting a lot of CPU cycles before. There's probably more improvements that could be made here, open to suggestion from those who understand C++ better. ---------------------------------------------------------------- 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: [email protected]
