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] Implement async GetFlightInfo #37093

Closed
Tracked by #34607
lidavidm opened this issue Aug 9, 2023 · 4 comments · Fixed by #36986
Closed
Tracked by #34607

[FlightRPC][Python] Implement async GetFlightInfo #37093

lidavidm opened this issue Aug 9, 2023 · 4 comments · Fixed by #36986
Assignees
@lidavidm
Labels
Component: FlightRPC Component: Python Type: enhancement
Milestone
14.0.0

Comments

@lidavidm
Copy link
Member

lidavidm commented Aug 9, 2023

Describe the enhancement requested

Add Python bindings to #36512.

Component(s)

FlightRPC, Python
@lidavidm lidavidm self-assigned this Aug 9, 2023
@lidavidm lidavidm mentioned this issue Aug 9, 2023
Open
18 tasks
@github-actions github-actions bot mentioned this issue Aug 9, 2023
Merged
lidavidm added a commit to lidavidm/arrow that referenced this issue Aug 9, 2023
@lidavidm
apacheGH-37093: [FlightRPC][Python] Add async GetFlightInfo
a89fe71
lidavidm added a commit to lidavidm/arrow that referenced this issue Aug 9, 2023
@lidavidm
apacheGH-37093: [FlightRPC][Python] Add async GetFlightInfo
68f2aa0
lidavidm added a commit to lidavidm/arrow that referenced this issue Aug 18, 2023
@lidavidm
apacheGH-37093: [FlightRPC][Python] Add async GetFlightInfo
3e05f9d
pitrou pushed a commit to lidavidm/arrow that referenced this issue Aug 23, 2023
@lidavidm @pitrou
apacheGH-37093: [FlightRPC][Python] Add async GetFlightInfo
adab985
pitrou added a commit that referenced this issue Aug 23, 2023
@lidavidm @pitrou
GH-37093: [Python] Add async Flight client with GetFlightInfo (#36986)
b6ab909 
### Rationale for this change

Demonstrate how to deal with async Flight from Python.

### What changes are included in this PR?

Add bindings for GetFlightInfo.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes, new APIs.

* Closes: #37093

Lead-authored-by: David Li <li.davidm96@gmail.com>
Co-authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@pitrou pitrou added this to the 14.0.0 milestone Aug 23, 2023
loicalleyne pushed a commit to loicalleyne/arrow that referenced this issue Nov 13, 2023
@lidavidm @pitrou @loicalleyne
apacheGH-37093: [Python] Add async Flight client with GetFlightInfo (a…
b20ec33 
…pache#36986)

### Rationale for this change

Demonstrate how to deal with async Flight from Python.

### What changes are included in this PR?

Add bindings for GetFlightInfo.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes, new APIs.

* Closes: apache#37093

Lead-authored-by: David Li <li.davidm96@gmail.com>
Co-authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@pvardanis
Copy link

pvardanis commented Jul 31, 2024 •
edited
Loading

@lidavidm any future plans to implement async Python bindings to do_exchange? I've already managed to process do_exchange calls concurrently as posted on another ticket earlier this year, but would love to have async functionality as well. I'm using Arrow Flight RPC for model inference, and this would be particularly useful for large model inference.

@lidavidm
Copy link
Member Author

Hey @pvardanis, unfortunately I ran out of available time after this. But since then I've come to the conclusion that we just can't feasibly wrap gRPC's API surface (async included) and that it would be better to expose gRPC so that you can directly use all of its APIs, which should mean that you wouldn't be blocked on us. CC @felipecrv who has been looking at this as well

@felipecrv
Copy link
Contributor

Yes. Giving users the full gRPC interface is my goal. I'm gonna have to do it sooner rather than later because it blocks other things I want to do.

@pvardanis
Copy link

@lidavidm @felipecrv has there been any roadmap for implementing this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lidavidm lidavidm

Labels
Component: FlightRPC Component: Python Type: enhancement
Projects
None yet
Milestone
14.0.0
Development

Successfully merging a pull request may close this issue.

GH-37093: [Python] Add async Flight client with GetFlightInfo lidavidm/arrow
4 participants
@felipecrv @lidavidm @pitrou @pvardanis