lidavidm commented on PR #3712: URL: https://github.com/apache/arrow-adbc/pull/3712#issuecomment-3519282038
@zeroshade we should look at Rust APIs as part of the current effort to add async APIs @eitsupi or @mbrobbel thoughts there? As for this PR: I think it looks OK, but I'm not an expert on Tokio. My main question is whether the driver should instead allow you to pass your own Handle in during construction so you can provide the async runtime explicitly? Otherwise I think falling back to the ambient runtime and only creating the runtime as a last resort makes sense -- 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]
