Skip to content
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

[FlightSQL][Go] Support for metadata requests #12496

Closed
stoffeastrom opened this issue Feb 23, 2022 · 4 comments · Fixed by #13828
Closed

[FlightSQL][Go] Support for metadata requests #12496

stoffeastrom opened this issue Feb 23, 2022 · 4 comments · Fixed by #13828

Comments

@stoffeastrom
Copy link

Hi 👋

Are there any plans to support metadata requests with the golang flight client❓

@zeroshade
Copy link
Member

Which metadata requests aren't currently supported? for record batch handling, both the reader and writer support handling the AppMetadata field of the message. Which request do you need that isn't yet supported?

@stoffeastrom
Copy link
Author

I was trying to find these

CommandGetCatalogs: list catalogs in a database.
CommandGetCrossReference: list foreign key columns that reference a particular other table.
CommandGetDbSchemas: list schemas in a catalog.
CommandGetExportedKeys: list foreign keys referencing a table.
CommandGetImportedKeys: list foreign keys of a table.
CommandGetPrimaryKeys: list primary keys of a table.
CommandGetSqlInfo: get information about the database itself and its supported SQL dialect.
CommandGetTables: list tables in a catalog/schema.
CommandGetTableTypes: list table types supported (e.g. table, view, system table).

after read the blog post
but couldn't find anything under go path in the repo. I'll look into the AppMetadata.

Thanks for your reply @zeroshade

@wjones127
Copy link
Member

@stoffeastrom Those are for FlightSQL, which is built on top of Flight RPC. FlightSQL is brand new to the Java and C++ implementations, and isn't implemented in Go yet.

@stoffeastrom stoffeastrom changed the title [Flight][Go] Support for metadata requests [FlightSQL][Go] Support for metadata requests Feb 25, 2022
@zeroshade
Copy link
Member

@stoffeastrom There are plans to support FlightSQL in Go eventually, but there are a few requirements that need to get implemented first such as Union arrays and such since those are used in FlightSQL for a variety of responses etc. I don't have any timeline i can give you for that unfortunately.

zeroshade added a commit that referenced this issue Aug 11, 2022
This implements initial support for FlightSQL in Golang and is sufficient to pass the flight_sql integration scenario.

As a follow-up in a subsequent PR i'll implement an equivalent example to the C++ SQLite example using Go.

Also closes #12496

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
ksuarez1423 pushed a commit to ksuarez1423/arrow that referenced this issue Aug 15, 2022
This implements initial support for FlightSQL in Golang and is sufficient to pass the flight_sql integration scenario.

As a follow-up in a subsequent PR i'll implement an equivalent example to the C++ SQLite example using Go.

Also closes apache#12496

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
ksuarez1423 pushed a commit to ksuarez1423/arrow that referenced this issue Aug 15, 2022
This implements initial support for FlightSQL in Golang and is sufficient to pass the flight_sql integration scenario.

As a follow-up in a subsequent PR i'll implement an equivalent example to the C++ SQLite example using Go.

Also closes apache#12496

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
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 a pull request may close this issue.

3 participants