Skip to content

[Java][FlightRPC] Flight SQL JDBC driver improvements #244

@asfimport

Description

@asfimport

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

Reporter: David Li / @lidavidm

Subtasks:

Note: This issue was originally created as ARROW-17729. Please see the migration documentation for further details.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions