Skip to content

Example for Python HTTP Post Arrow client and server#27

Closed
arunppsg wants to merge 4 commits into
apache:mainfrom
arunppsg:py-http-post-gh-40595
Closed

Example for Python HTTP Post Arrow client and server#27
arunppsg wants to merge 4 commits into
apache:mainfrom
arunppsg:py-http-post-gh-40595

Conversation

@arunppsg

Copy link
Copy Markdown
Contributor

Addresses apache/arrow#40595

In this pull request, I have added an example of HTTP Post Arrow client and server.

The server uses fastapi for and the client uses requests library. Except these, the rest of the code structure is similar to the get_simple example.

If there are any suggestions, feel free to let me know and I will be happy to update the PR accordingly.

@ianmcook ianmcook self-requested a review March 27, 2024 16:24
@ianmcook

Copy link
Copy Markdown
Member

Thank you @arunppsg! I will review this soon.

@ianmcook

ianmcook commented Mar 28, 2024

Copy link
Copy Markdown
Member

@arunppsg For the HTTP POST example, we are looking for something different than this.

As described in the README in the http/post_simple directory:

This directory contains minimal examples of HTTP clients and servers demonstrating how a client can send a POST request to a server with a request body consisting of an Arrow IPC stream of record batches.

So in these HTTP POST examples, the client should send the data to the server.

However, I think this FastAPI + Uvicorn example would be a great addition to the HTTP GET examples. May I make some changes to this PR to turn it into an HTTP GET example?

@arunppsg

Copy link
Copy Markdown
Contributor Author

So in these HTTP POST examples, the client should send the data to the server.

Ah, I see - I will think about this and get back on it.

However, I think this FastAPI + Uvicorn example would be a great addition to the HTTP GET examples. May I make some changes to this PR to turn it into an HTTP GET example?

Feel free to make the necessary changes for HTTP GET in this PR.

@ianmcook

Copy link
Copy Markdown
Member

I moved the FastAPI + Uvicorn server piece of this to #37 (with some small changes).

@ianmcook ianmcook closed this Sep 13, 2024
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 this pull request may close these issues.

2 participants