arrow icon indicating copy to clipboard operation
arrow copied to clipboard

ARROW-15452: [FlightRPC][Java] JDBC driver for Flight SQL

Open rafael-telles opened this issue 2 years ago • 11 comments

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

rafael-telles avatar Jan 25 '22 14:01 rafael-telles

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:

github-actions[bot] avatar Jan 25 '22 14:01 github-actions[bot]

https://issues.apache.org/jira/browse/ARROW-15452

github-actions[bot] avatar Jan 25 '22 16:01 github-actions[bot]

:warning: Ticket has not been started in JIRA, please click 'Start Progress'.

github-actions[bot] avatar Jan 25 '22 16:01 github-actions[bot]

Thank you so much @lidavidm ! We will be working on your feedback soon

rafael-telles avatar Jan 26 '22 22:01 rafael-telles

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.

wesm avatar Feb 04 '22 23:02 wesm

Thanks for chiming in Wes - I'll look into the process and follow up on the mailing list @rafael-telles.

lidavidm avatar Feb 04 '22 23:02 lidavidm

BTW, please let us know when you think this is ready for a full review

lidavidm avatar Mar 10 '22 21:03 lidavidm

This can be rebased against the accepted proposals now.

lidavidm avatar Mar 25 '22 21:03 lidavidm

For the IP clearance (moved to #12830)

lidavidm avatar Apr 05 '22 15:04 lidavidm

@rafael-telles It seems this needs rebasing on git master and then fixing conflicts, would you like to do that?

pitrou avatar Jun 07 '22 11:06 pitrou

We should probably close this PR in favor of a new PR based on the cleared code

lidavidm avatar Aug 03 '22 19:08 lidavidm

Closing because it has been untouched for a while, in case it's still relevant feel free to reopen and move it forward 👍

amol- avatar Mar 30 '23 17:03 amol-