Re: [FlightSQL][JDBC] Additional changes to the JDBC driver

2022-08-05 Thread David Li
We did merge the cleared IP into a branch: https://github.com/apache/arrow/tree/flight-sql-jdbc James then took it and filed a PR, which we can review: https://github.com/apache/arrow/pull/13800 So long as further changes build on top of that commit, I guess we can just include them in the exis

Re: [FlightSQL][JDBC] Additional changes to the JDBC driver

2022-08-05 Thread Wes McKinney
If you want to merge the cleared IP into a new branch rather than master, that is fine, too. It's not necessary to land it in the main branch On Tue, Aug 2, 2022 at 4:18 PM David Li wrote: > > Would it be OK to get what's there into the main branch first? i.e., open a > PR from the apache/flight

Re: [FlightSQL][JDBC] Additional changes to the JDBC driver

2022-08-02 Thread David Li
Would it be OK to get what's there into the main branch first? i.e., open a PR from the apache/flight-jdbc-driver (or a contributor's clone of it, that would make it easier to address review comments). I'd like to get through the review of what we currently have since the PR will be large. And t

[FlightSQL][JDBC] Additional changes to the JDBC driver

2022-08-02 Thread James Duong
Hi, We have a few additional important changes for the Flight SQL JDBC Driver: - Avoid sending headers for built-in properties such as hostname, port. - Make handling of connection URI key names and Properties keys case-insensitive. - Create separate FlightClients for each new endpoint returned by