[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-1456817950

   @srebhan 
   
   > Unification of backends: I think I should not deal with backend-specific settings at the driver level. So my idea is to unify those and let the user do the specifics on the DSN level. I hope to finish this next Friday...
   
   Agreed, unifying the backends would be good and is kind of the point of using flightsql in general i think.
   
   > TLS implementation/customization: My plan is to implement a mechanism similar to the [MySQL driver](https://github.com/go-sql-driver/mysql) to have a driver-global TLS-config registry. I thought about doing this in a separate PR once this one is merged. What do you think?
   
   I haven't looked at that too hard but I think it's fine to do it as a separate PR once this is merged.


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