itsjunetime opened a new pull request, #6612: URL: https://github.com/apache/arrow-rs/pull/6612
# Which issue does this PR close? Closes #6248 # What changes are included in this PR? This PR works on top of #6249 to add a test for the new `with_runtime` fn, as well as changing the signature of the `spawn` function slightly to avoid an extra re-boxing when a runtime is set. This also fixes a few things that `clippy` was complaining about. # Rationale for this change See #6248 for the API addition. With regard to the test, I felt like this is really the only test we'd want for this feature - we just want to make sure that the runtime is actually being used by `ParquetObjectReader`. We can't make any guarantees about how it actually performs or would work if there's another runtime being used for CPU-bound operations, so all we really want to test is if it is used. # 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]
