felipecrv commented on issue #2809: URL: https://github.com/apache/arrow-adbc/issues/2809#issuecomment-2876444252
> ...needs to be not mutable to fit in to some framework? The framework is Rust itself. 😄 When you have a `&mut` you can't have other `&`s. The `adbc_core` objects wrap a `Mutex<FFI_Blah>` so they are internally mutable. If you require the wrapper to also be mutable I now have to wrap the wrapper in a `Mutex<Blah>`. For connections, I think preventing use from multiple-threads is a good thing because connections are mutable accumulators of stataments that form a transaction and order in which these statements reach the connection matters a lot. For connections, instead of wrapping them in a Mutex, I have one connection per thread (thread-confinement strategy for thread-safety). -- 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]
