nealrichardson opened a new pull request #9209: URL: https://github.com/apache/arrow/pull/9209
Key features: * if arrowExports.* are not modified, they are not rewritten. This means that if there are no cpp changes, then nothing gets recompiled, which makes local dev faster (when you have ARROW_R_DEV=true set) * refactoring of codegen.R to be more consistently aware of "features" that can be flagged, like we have currently for `arrow` and `s3` and potentially may get more of in the future with additional build-time features. Importantly, the wrapping with the fake implementation that we currently have for all functions has been encapsulated in such a way that we can choose whether to make some features always on or not. At this point, this PR is purely a refactor, but by uncommenting an `if` statement, we can effectively remove the arrow-without-arrow wrapping while still keeping S3 support optional. ---------------------------------------------------------------- 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]
