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

[WIP] ARROW-4463: [Rust] Add Feather read:write support #3549

Closed
wants to merge 2 commits into from

Conversation

nevi-me
Copy link
Contributor

@nevi-me nevi-me commented Feb 3, 2019

Adds the ability to convert feather files to and from record batches.

Adds the ability to convert feather files to and from record batches.
@nevi-me
Copy link
Contributor Author

nevi-me commented Feb 3, 2019

@andygrove @sunchao @paddyhoran PTAL when convenient, I'm largely looking for review and guidance on the API, and parts of the code that I can DRY.

@wesm I looked at the Feather repo, as well as within Arrow to see whether Feather is still supported or should be regarded as deprecated. I'm still unsure of what the case is.

I know that there's Arrow File format in IPC, but I opted to start with Feather (with the intent that we'd be able to reuse most of the code when implementing Arrow RPC in Rust) as I had an immediate need for being able to write Arrow files to memory.

@wesm
Copy link
Member

wesm commented Feb 3, 2019

I responded on the JIRA. The internal details of the current Feather format should be regarded as deprecated. I would recommend trying to repurpose as much of your work here in supporting the more general Arrow IPC protocol. If you want to maintain support for the legacy Feather format for now that's your choice ;)

@nevi-me nevi-me closed this Feb 4, 2019
@nevi-me nevi-me deleted the rust/feather branch March 4, 2019 19:48
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.

None yet

2 participants