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

Update Flight proto #5367

Closed
Jefffrey opened this issue Feb 6, 2024 · 3 comments · Fixed by #5413
Closed

Update Flight proto #5367

Jefffrey opened this issue Feb 6, 2024 · 3 comments · Fixed by #5413
Assignees
Labels
arrow Changes to the arrow crate arrow-flight Changes to the arrow-flight crate enhancement Any new improvement worthy of a entry in the changelog

Comments

@Jefffrey
Copy link
Contributor

Jefffrey commented Feb 6, 2024

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

From mailing list: https://lists.apache.org/thread/4t664cj5z52brmbshjb6gknfq30ckhcn

Now, here lies the issue: neither FlightEndpoint nor the FlightInfo message in the arrow.flight.protocol.rs file of arrow-rs includes an 'app_metadata' field (It seems outdated). Consequently, I am currently facing a roadblock.
Is there a plan to update the flight implementation of arrow-rs?

Need to update the Flight & FlightSql proto as they are outdated.

See arrow source:

Describe the solution you'd like

Update format files and generated code (and interop code, refer to PR below for previous example)

Describe alternatives you've considered

Additional context

For reference, last update PR: #4250

@Jefffrey Jefffrey added the enhancement Any new improvement worthy of a entry in the changelog label Feb 6, 2024
@Jefffrey
Copy link
Contributor Author

Jefffrey commented Feb 6, 2024

I can take a stab at this next week or so, unless someone else is interested in picking this up 👀

@Jefffrey Jefffrey self-assigned this Feb 15, 2024
@tustvold tustvold added the arrow Changes to the arrow crate label Mar 1, 2024
@tustvold
Copy link
Contributor

tustvold commented Mar 1, 2024

label_issue.py automatically added labels {'arrow'} from #5413

@tustvold tustvold added the arrow-flight Changes to the arrow-flight crate label Mar 1, 2024
@tustvold
Copy link
Contributor

tustvold commented Mar 1, 2024

label_issue.py automatically added labels {'arrow-flight'} from #5413

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrow Changes to the arrow crate arrow-flight Changes to the arrow-flight crate enhancement Any new improvement worthy of a entry in the changelog
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants