unikdahal opened a new pull request, #4747:
URL: https://github.com/apache/arrow-adbc/pull/4747
## Summary
Add Flight SQL session management to the Java ADBC driver.
This supersedes #4444, rebuilt from current `main` as a clean single
commit.
- Add typed session option get, set, and erase support
- Add `GetSessionOptions`, `SetSessionOptions`, and `CloseSession`
- Add JSON serialization and validation
- Add unit and edge-case tests
## Changes since the previous review
- Following @Kyperr’s feedback, session RPCs preserve connection-level
call options through `FlightSqlClientWithCallOptions`. The explicit
`callOptions` argument was removed because the wrapper already applies
them, avoiding duplicate options.
- Following @lidavidm’s feedback, `CloseSession` is included in
`AutoCloseables.close` as a selectively best-effort lambda.
- Non-finite doubles are converted inside the JSON visitor and represented
as strings.
- Added edge-case coverage.
Thanks @lidavidm and @Kyperr for reviewing the previous version.
Closes #4443.
Closes #2821.
Relates to #745.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
