-
Notifications
You must be signed in to change notification settings - Fork 198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update documentation to reflect the release of the FlightSQL JDBC Driver #461
Conversation
@andygrove heads up if you want to get this in for the blog post. |
docs/source/user-guide/flightsql.md
Outdated
@@ -103,6 +92,9 @@ The important pieces of information: | |||
| Advanced Options | useEncryption=false | | |||
| URL | jdbc:arrow-flight://127.0.0.1:50050 | | |||
|
|||
**Note**: The `useEnryption` flag will not be required if [this PR](https://github.com/spaceandtimelabs/arrow-ballista/pull/15) is merged, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason we can't have this PR open against the apache/arrow-ballista
repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be @andygrove . Probably @iajoiner opened it before the FlightSQL changes from our fork were upstreamed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will close that one and open a new one.
Co-authored-by: Andy Grove <[email protected]>
Which issue does this PR close?
Closes #460.
Rationale for this change
Described in the issue.
What changes are included in this PR?
Are there any user-facing changes?
Using JDBC will be easier, so more people will be likely to try out Ballista