GH-49723: [C++][FlightRPC][ODBC] Update ODBC Documentation#49851
GH-49723: [C++][FlightRPC][ODBC] Update ODBC Documentation#49851justing-bq wants to merge 3 commits intoapache:mainfrom
Conversation
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or See also: |
|
|
c9d55a8 to
8e68507
Compare
| | `host` | The IP address or hostname for the server. | `NONE` | ||
| | `port` | The TCP port number the server uses for ODBC connections. | `NONE` | ||
| | `user` | The username for authentication to the server. | `NONE` | ||
| | `user id` | The username for authentication to the server. | `NONE` |
There was a problem hiding this comment.
This includes a space. Is it correct?
There was a problem hiding this comment.
Yes this is one of the keys accepted by the driver.
There was a problem hiding this comment.
Pull request overview
Updates documentation for the Arrow Flight SQL ODBC driver to better support Unix/macOS users ahead of public release (GH-49723), including a new reference for connection properties and expanded install/registration steps across platforms.
Changes:
- Add a new
connection-options.mddocument describing connection string / DSN properties. - Update ODBC README with Windows/macOS/Linux driver binary locations and registration commands.
- Add a Linux-specific known limitation note for
isql.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 14 comments.
| File | Description |
|---|---|
cpp/src/arrow/flight/sql/odbc/connection-options.md |
New doc describing ODBC connection properties and defaults. |
cpp/src/arrow/flight/sql/odbc/README.md |
Expanded registration instructions for macOS/Linux and updated known limitations. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f132729 to
99bb351
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
99bb351 to
96f38fd
Compare
Rationale for this change
Update ODBC Documentation in preparation for public release.
Resolves #49723
What changes are included in this PR?
Miscellaneous doc changes.
Are these changes tested?
N/A
Are there any user-facing changes?
No