[JDBC] Honor DriverManager login timeout
- Ngôn ngữ chính
- Java
- Star
- 94
- Fork
- 152
- Merge trung bình
- 3 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 11
Mô tả
### Describe the enhancement requested
`ArrowFlightJdbcDriver.connect()` does not currently honor the timeout configured through `DriverManager.setLoginTimeout(int)`.
Implement support for the `DriverManager` login timeout:
- Read the value exposed by `DriverManager.getLoginTimeout()` when establishing a connection selected by `DriverManager`.
- Preserve the JDBC semantics of zero: do not impose a login timeout.
- Apply a nonzero timeout to the overall connection attempt, not only the underlying Netty socket connection.
- If the timeout expires, attempt to cancel the connection attempt and throw `SQLTimeoutException`.
- Clean up any allocator, channel, or partially constructed client when the attempt is cancelled or times out.
- Add tests that set and restore the global `DriverManager` timeout and verify timeout enforcement, exception type, and resource cleanup.
This is distinct from #494/#495. The existing `connectTimeoutMs` property configures Netty socket connection attempts and does not consume the timeout configured through `DriverManager`.
Context: #1027 and [the follow-up discussion](https://github.com/apache/arrow-java/pull/1027#issuecomment-5447203124).
Related DataSource work: https://github.com/apache/arrow-java/issues/1267
Hướng dẫn đóng góp
Hướng nghiên cứu
Điểm vào là ArrowFlightJdbcDriver.connect(); trước tiên hãy đọc đường dẫn kết nối hiện tại của nó và cách xử lý connectTimeoutMs hiện có. Bổ sung coverage để thiết lập và khôi phục timeout toàn cục của DriverManager, kiểm tra hành vi với timeout bằng 0 và khác 0, SQLTimeoutException, cũng như việc dọn dẹp allocator, channel và client khi toàn bộ lần thử bị hủy hoặc hết thời gian.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- Lĩnh vực
- backend, database
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Sôi nổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 55/100