jonkeane commented on PR #42241: URL: https://github.com/apache/arrow/pull/42241#issuecomment-2209249897
> It wasn't easy to expose the writer's closed_ state to the R package for Arrow C++ 13/14 so I opted just to skip the test on Arrow C++ <15, see https://github.com/apache/arrow/commit/22a706b2e2d5eaf7cbacbbcf6de6a9cfa2132a09. @jonkeane can you take a quick look? Does this mean we should bump our minimum libarrow to 15 then? If we can't support it, that's the way to go IMO. The backwards compatibility with libarrow is nice, but as far as I know it's not used anywhere specifically. It was added with the hopes that we might use system-library libarrow on CRAN among other places, but that hasn't happened. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
