[GitHub] [arrow] zeroshade commented on pull request #34331: GH-34332: [Go][FlightRPC] Add driver for `database/sql` framework

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade commented on PR #34331:
URL: https://github.com/apache/arrow/pull/34331#issuecomment-1443901087

   @srebhan Would it make more sense to add IOx backend testing to the CI for adbc/sqldriver (as pointed to by @lidavidm) along with any necessary changes there?
   
   At the moment we're trying to keep the arrow module/package compatible with go1.17+ so at a minimum you'd need to add `go:build` constraints to not break the CI here. But it would probably be better to consolidate efforts in the ADBC sqldriver (which would provide a `database/sql` wrapper for *anything* that implements ADBC including the FlightSQL driver) rather than duplicating a large part of that work here.
   
   What do you think?


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org