[ARROW-18296] [Java][FlightRPC] FlightSQL JDBC precludes other JDBC drivers on the same JVM

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 10.0.0
• Fix Version/s: 10.0.1
• Component/s: FlightRPC, Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/33472
• Assignee: David Li
• Reporter: James Henderson
• Created: 09/Nov/22 14:40
• Updated: 11/Jan/23 11:59
• Resolved: 10/Nov/22 00:59

Description

The contract of `java.sql.Driver/connect` says that the driver should return null if it realises it's the wrong kind of driver to handle the connection - unfortunately the override in `ArrowFlightJdbcDriver` throws an exception if the scheme isn't arrow-flight or arrow-flight-sql. So, if I'm trying to connect to (e.g.) postgres and the Arrow Flight driver happens to be first in the java.sql.DriverManager's list, I'll get a URL Scheme must be 'arrow-flight'. error.

We could consider a similar pattern to Avatica's UnregisteredDriver/connect, which checks `acceptsURL` first and returns null if appropriate?

(Separately, I suspect that error message should say arrow-flight-sql, given arrow-flight seems to be deprecated?)

Cheers,

James

Issue Links

• links to: GitHub Pull Request #14617