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

[C++] Implement DoPut command for Flight on client and server side #19627

Closed
asfimport opened this issue Sep 21, 2018 · 7 comments
Closed

[C++] Implement DoPut command for Flight on client and server side #19627

asfimport opened this issue Sep 21, 2018 · 7 comments

Comments

@asfimport
Copy link
Collaborator

This was omitted from ARROW-3146

Reporter: Wes McKinney / @wesm
Assignee: David Li / @lidavidm

PRs and other links:

Note: This issue was originally created as ARROW-3289. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I can take on this next, seeing as I've been digging around the relevant parts anyways, and then update the integration tests to cover uploads.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
That sounds great – the Java test server supports storing a dataset, then retrieving it. I figure you would implement similar functionality in order to test this

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
Yup - the current integration tests I have written write the data back out to a file, but with the put side implemented, it can/should just stay in memory.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I have an initial implementation ready now; waiting on ARROW-4213 to be reviewed first though.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
I'll review ARROW-4213 today so we can get that merged soon, thank you for your patience

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
No rush, thank you for your time!

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Issue resolved by pull request 3524
#3524

@asfimport asfimport added this to the 0.13.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants