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.

Sign up for GitHub

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

Jump to bottom

[FlightRPC][Python] Expose new Flight features in Python #36954

Closed
7 tasks
lidavidm opened this issue Jul 31, 2023 · 4 comments
Closed
7 tasks

[FlightRPC][Python] Expose new Flight features in Python #36954

lidavidm opened this issue Jul 31, 2023 · 4 comments
Labels
Component: FlightRPC Component: Python Type: enhancement
Milestone
19.0.0

Comments

@lidavidm
Copy link
Member

lidavidm commented Jul 31, 2023 •
edited
Loading

Describe the enhancement requested

These recent Flight RPC improvements weren't exposed to Python:

Component(s)

FlightRPC, Python
@lidavidm lidavidm added this to the 14.0.0 milestone Jul 31, 2023
@lidavidm lidavidm self-assigned this Jul 31, 2023
@lidavidm lidavidm mentioned this issue Jul 31, 2023
Merged
@jorisvandenbossche jorisvandenbossche modified the milestones: 14.0.0, 15.0.0 Oct 5, 2023
@jorisvandenbossche
Copy link
Member

(moving to 15.0)

@lidavidm lidavidm removed their assignment Oct 5, 2023
@lidavidm lidavidm removed this from the 15.0.0 milestone Oct 5, 2023
@lidavidm
Copy link
Member Author

lidavidm commented Oct 5, 2023

I'm going to dump it since I don't think I'll have any time for the forseable feature

@lidavidm lidavidm mentioned this issue Jun 11, 2024
Closed
@felipemmp
Copy link

Hi, are there any plans to implement this in the short term? And if not, are there any workarounds for me to retrieve the data in these fields?

@github-actions github-actions bot mentioned this issue Aug 2, 2024
Merged
lidavidm added a commit that referenced this issue Nov 7, 2024
@EnricoMi @adamreeve
@jorisvandenbossche @lidavidm
GH-36954: [Python] Add more FlightInfo / FlightEndpoint attributes (#…
b193c4f 
…43537)

### Rationale for this change
The C++ classes `FlightInfo` and `FlightEndpoint` have attributes that are not available via the Python API.

### What changes are included in this PR?
Make the following attributes available in Python:
- `FlightInfo.ordered`
- `FlightInfo.app_metadata`
- `FlightEndpoint.expiration_time`
- `FlightEndpoint.app_metadata`

Also makes existing attributes optional in constructor:
- `FlightInfo.total_records`
- `FlightInfo.total_bytes`

### Are these changes tested?
Existing tests that test existing attributes are extended.

### Are there any user-facing changes?
Yes, changes are backward compatible.
* GitHub Issue: #36954

Lead-authored-by: Enrico Minack <github@enrico.minack.dev>
Co-authored-by: Adam Reeve <adreeve@gmail.com>
Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Co-authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
@lidavidm lidavidm added this to the 19.0.0 milestone Nov 7, 2024
@lidavidm
Copy link
Member Author

lidavidm commented Nov 7, 2024

Issue resolved by pull request 43537
#43537

@lidavidm lidavidm closed this as completed Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: FlightRPC Component: Python Type: enhancement
Projects
None yet
Milestone
19.0.0
Development

No branches or pull requests
3 participants
@lidavidm @jorisvandenbossche @felipemmp