[ARROW-17729] [Java][FlightRPC] Flight SQL JDBC driver improvements

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: FlightRPC, Java
• External issue URL: https://github.com/apache/arrow/issues/32964
• Assignee: Unassigned
• Reporter: David Li
• Created: 14/Sep/22 18:11
• Updated: 11/Jan/23 11:52

Description

Follow ups for ARROW-7744

• Rename internal classes to not imply everything is part of Flight RPC (e.g. ArrowFlightJdbcArray -> FieldVectorArray or similar)
• Don't throw bare exceptions (always provide some error context)
• Log a warning if the arrow-flight: URI scheme is used instead of arrow-flight-sql:
• Create a documentation page (that can be used for people approaching this from the JDBC side, not necessarily Arrow users)
• Document the connection string format and parameters
• Replace // TODO comments with throw new UnsupportedOperationException()
• Document how timestamp/time/date types are handled in converting between the two type schemas
• Document the type conversions in general
• timestamp handling is suspect
• Upgrade to JUnit5/AssertJ instead of JUnit4/Hamcrest
• Get rid of FreePortFinder
• Use a single (static?) RootAllocator with child allocators
• Remove all Guava usage (can we deny this with a lint?)
• See if SqlTypes can be folded into arrow-jdbc

Sub-Tasks

• [Docs][Java] Add documentation page for Flight SQL JDBC driver: Resolved
• [Packaging] Add JDBC driver to release tasks: Resolved