apache / apache/arrow-java

[Java][FlightRPC] Flight SQL JDBC driver improvements

未关闭
#244 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Type: enhancement
主要语言
Java
星标
94
派生
152
平均合并
3 天 16 小时
30 天内合并 PR
11

描述

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](https://github.com/apache/arrow/pull/13800#discussion_r938908230)
- 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](https://issues.apache.org/jira/browse/ARROW-17729) / @lidavidm
#### Subtasks:
- [X] [[Docs][Java] Add documentation page for Flight SQL JDBC driver](https://github.com/apache/arrow/issues/32967)
- [X] [[Packaging] Add JDBC driver to release tasks](https://github.com/apache/arrow/issues/32976)

**Note**: *This issue was originally created as [ARROW-17729](https://issues.apache.org/jira/browse/ARROW-17729). Please see the [migration documentation](https://github.com/apache/arrow/issues/14542) for further details.*

贡献指南

打开贡献指南

调研方向

首先将剩余的后续工作与已完成的两个子任务分开,然后检查 issue 中指出的 Flight SQL JDBC driver 入口点,并确定哪些未勾选项仍然适用。完成意味着已完成所选的改进、文档、测试和清理工作,同时不让更广泛的 issue 继续处于含糊状态。

由索引模型根据 Issue 内容生成。

评估

技术栈
java, sql
领域
databases
Issue 类型
重构
难度
5/5
预计耗时
一周以上
活跃度
停滞
描述清晰度
需要澄清
新手友好度
25/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。