jadewang-db commented on PR #2998: URL: https://github.com/apache/arrow-adbc/pull/2998#issuecomment-2988667280
> I'll give this a full review tomorrow, but it looks like you're wrapping something that uses the `database/sql` API, it might make more sense to just have a generic adapter for doing that instead of something `Databricks` specific? I can do that if possible, but likely we will need some extension, because seems the database/sql is not arrow based, in order to make this an performant driver, it's better to use arrow directly. maybe extend the database/sql to have arrow functionality. I am not a go expert, suggestion welcomed. -- 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]
