arrow
arrow copied to clipboard
ARROW-15452: [FlightRPC][Java] JDBC driver for Flight SQL
This implements a JDBC driver able to communicate to Flight SQL sources.
So far this covers:
- Metadata retrieval by DatabaseMetadata, ResultSetMetadata, etc.
- Query execution by statements and prepared statements
Yet to be done:
- Parameter binding on prepared statements
Thanks for opening a pull request!
If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW
Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project.
Then could you also rename pull request title in the following format?
ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}
or
MINOR: [${COMPONENT}] ${SUMMARY}
See also:
https://issues.apache.org/jira/browse/ARROW-15452
:warning: Ticket has not been started in JIRA, please click 'Start Progress'.
Thank you so much @lidavidm ! We will be working on your feedback soon
I just wanted to echo my comment from the mailing list that I think it would be best if we held a code donation vote and IP clearance for this piece of work since it is substantial and was developed outside of the Arrow community. I don't expect there will be any issues but it is good due diligence. As a reminder we welcome the development of large new projects in feature branches, where the development process otherwise is conducted according to the Apache Way (public discussions, issues, PRs, etc.)
I believe this will involve a Software Grant from Dremio (correct me if I'm wrong) for the work and ICLAs for the individual contributors.
Thanks for chiming in Wes - I'll look into the process and follow up on the mailing list @rafael-telles.
BTW, please let us know when you think this is ready for a full review
This can be rebased against the accepted proposals now.
For the IP clearance (moved to #12830)
@rafael-telles It seems this needs rebasing on git master and then fixing conflicts, would you like to do that?
We should probably close this PR in favor of a new PR based on the cleared code
Closing because it has been untouched for a while, in case it's still relevant feel free to reopen and move it forward 👍