jorisvandenbossche commented on pull request #7272: URL: https://github.com/apache/arrow/pull/7272#issuecomment-656105473
I updated this, and added a small C++ test. > Are you intending to make that change in this PR too? Realistically speaking, not at the moment (I certainly want to learn how to set up a R dev environment, but just before the release with other priorities might not be the best time ;-)) So would either merge without, unless you can quickly push a change (I suppose it might be an easy change). ---------------------------------------------------------------- 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