Skip to content

Conversation

@lidavidm
Copy link
Member

  • Adds more connection options around TLS.
  • Adds basic smoke tests for connection options.
  • Error on unknown options.
  • Serialize partitions as FlightInfo with one FlightEndpoint. This is verbose, but allows preserving any other metadata that may be added to Flight in the future.

Part of #380.

@lidavidm lidavidm added this to the ADBC Libraries 0.2.0 milestone Jan 24, 2023
@lidavidm lidavidm requested a review from zeroshade January 24, 2023 16:58
@lidavidm
Copy link
Member Author

Eventually, I plan to port the C++ unit tests from apache/arrow#14082 into this repository to test the Go driver. But native Go tests are preferable since those are much easier to develop with.

Co-authored-by: Matt Topol <zotthewizard@gmail.com>
Copy link
Member

@zeroshade zeroshade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zeroshade zeroshade merged commit bc29a37 into apache:main Jan 24, 2023
@lidavidm lidavidm deleted the flightsql-config branch January 24, 2023 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants