mbrobbel commented on PR #2713: URL: https://github.com/apache/arrow-adbc/pull/2713#issuecomment-2813149682
> Is adding one dedicated type per function that returns a record batch reader sensible? We could reduce this to one associated type for all the returned readers? > If an ADBC driver can be used from languages like C++ and Python, that don't have statically checked lifetimes, it should be possible to use from Rust without all this complexity. > If you don't want to trust the ADBC driver implementations to handle the reference counting and weak references to connection (I would) you can keep an inner Arc inside every Connection and avoid trying to track this complex relationship statically. Is what you are describing provided by the driver manager? https://github.com/apache/arrow-adbc/blob/7f1dfca741c298209e213d00d785c7a220bdc0bb/rust/core/src/driver_manager.rs#L38-L41 -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
