Skip to content
This repository has been archived by the owner on Mar 5, 2024. It is now read-only.

FlightSQL Integration test #4

Open
avantgardnerio opened this issue Sep 14, 2022 · 4 comments
Open

FlightSQL Integration test #4

avantgardnerio opened this issue Sep 14, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@avantgardnerio
Copy link

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

FlightSQL support has been added to Ballista, but unfortunately it is hard to test because the only way (I know how) is using a custom fork of the FlightSQL JDBC driver. In order to ensure regressions don't happen, it would be good to test in an automated way.

Describe the solution you'd like

Automated integration testing between Java/JDBC and Rust would be ideal, but would add a lot of complexity. Therefor, it would be good to add a Rust FlightSQL client -> Ballista FlightSQL server test. I would recommend starting with a manual JDBC test to see how it is supposed to work, then emulating that test in pure Rust. The code-under-test is located here.

Describe alternatives you've considered

  1. A java/rust integration test with docker-compose in CI
  2. Manual testing
@avantgardnerio avantgardnerio added the enhancement New feature or request label Sep 14, 2022
@DhamoPS DhamoPS self-assigned this Sep 14, 2022
@avantgardnerio
Copy link
Author

I think we want a client here: https://github.com/apache/arrow-rs/tree/master/arrow-flight/src/sql (and an automated test too!)
(https://spaceandtimeworkspace.slack.com/archives/D03RMCRMTB5/p1664292652688569)

To satisfy this issue, we only need to do:

get_flight_info_statement("select 'hello, world!";)

@avantgardnerio
Copy link
Author

It appears this is a good starting point: wangfenjin/arrow-rs@7da031d

@avantgardnerio
Copy link
Author

@DhamoPS should we close this issue as a duplicate of apache/arrow-rs#1413 ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants