felipecrv commented on code in PR #3714:
URL: https://github.com/apache/arrow-adbc/pull/3714#discussion_r2620211532


##########
rust/core/src/non_blocking/local.rs:
##########


Review Comment:
   > This is about flexibility.
   
   We might end up offering too much flexibility in a standard that is supposed 
to make database access more uniform. If a database doesn't allow its handles 
to move between threads, it's the responsibility of the ADBC driver wrapping 
these database APIs to fix for that rather than bubbling up these restrictions 
to Rust users of ADBC via the type system which basically means a rewrite of 
the entire application depending on the ADBC driver that it uses.



-- 
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]

Reply via email to