paleolimbot opened a new pull request, #37274: URL: https://github.com/apache/arrow/pull/37274
### Rationale for this change When we first added RunWithCapturedR to support reading files from R connections, none of the Parquet tests seemed to call R from another thread. Because RunWithCapturedR comes with some complexity, I didn't add it anywhere it wasn't strictly needed. A recent StackOverflow post exposed that reading very large parquet files do use multiple threads and thus need RunWithCapturedR. ### What changes are included in this PR? The two most common calls to read a parquet in which a user might trigger this failure are now wrapped in RunWithCapturedR. ### Are these changes tested? The changes are tested in the current suite. ### Are there any user-facing changes? No. -- 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]
