lidavidm commented on PR #702: URL: https://github.com/apache/arrow-adbc/pull/702#issuecomment-1561189101
> Currently the DBAPI methods like execute(..) already initializes the pyarrow RecordBatchReader. We might want to delay that creation until actually fetch_arrow_table() gets called? (or one of the other fetch variants that and up consuming the RecordBatchReader as well) The slight concern there is then you wouldn't get any errors until you actually go to fetch data (this can be confusing; gRPC does this and it trips me up still), but this is probably OK -- 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]
