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

Migrate Flight and FlightSql Proto Gen #89

Open
joellubi opened this issue Sep 7, 2024 · 2 comments
Open

Migrate Flight and FlightSql Proto Gen #89

joellubi opened this issue Sep 7, 2024 · 2 comments

Comments

@joellubi
Copy link
Member

joellubi commented Sep 7, 2024

Flight and FlightSql definitions are still in apache/arrow.

The go_package name will need to be updated to match this repo, and the actual codegen will need to be ported as well. We will no longer be able to use a direct reference to the .proto files, so we may need to use git submodules or a tool like buf in order to build the needed files.

@kou
Copy link
Member

kou commented Sep 10, 2024

Package name change in apache/arrow: apache/arrow#44035

@kou
Copy link
Member

kou commented Sep 10, 2024

How about referring apache/arrow's format/ via an environment variable?
#114

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.

2 participants