paleolimbot commented on PR #13635: URL: https://github.com/apache/arrow/pull/13635#issuecomment-1256620157
I think this is clean and ready! In addition to the cancellation, there are a few improvements to the safe call into R infrastructure that were exposed in the process of implementing this. Notably, `RunWithCapturedR()` no longer throws an exception (which was causing at least one memory leak in the safe call into R tests). -- 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]
