-
Notifications
You must be signed in to change notification settings - Fork 96
Open
Labels
Type: enhancementNew feature or requestNew feature or request
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 ofarrow-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
// TODOcomments withthrow 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
SqlTypescan be folded into arrow-jdbc
Reporter: David Li / @lidavidm
Subtasks:
- [Docs][Java] Add documentation page for Flight SQL JDBC driver
- [Packaging] Add JDBC driver to release tasks
Note: This issue was originally created as ARROW-17729. Please see the migration documentation for further details.
Metadata
Metadata
Assignees
Labels
Type: enhancementNew feature or requestNew feature or request