wesm / feather

Feather: fast, interoperable binary data frame storage for Python, R, and more powered by Apache Arrow
Apache License 2.0
2.74k stars 168 forks source link

[R] Import arrow package and delete C++ implementation here #388

Closed nealrichardson closed 4 years ago

nealrichardson commented 4 years ago

For consideration, here's a PR that makes the feather R package import arrow, like how the Python implementation is just a wrapper around pyarrow. Aside from reducing maintenance burden, this will help us ensure that the Feather implementation in the arrow package meets all (tested) expectations of the feather package. Indeed, there were a few minor features missing from arrow version 0.17 that this exposed, and they were all addressed in the recent 1.0.0 release.

Most feather APIs are preserved in spirit, but key changes in this PR:

nealrichardson commented 4 years ago

Appveyor fails because I deleted .appveyor.yml. It was only testing the C++ code, which I deleted.