-
Notifications
You must be signed in to change notification settings - Fork 3.9k
GH-47472: [Doc] Add third-party implementations section #47488
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
base: main
Are you sure you want to change the base?
Conversation
Add a new section listing third-party open source implementations of Apache Arrow including cuDF, Sparrow, and arro3 as discussed in the issue.
|
Thanks for the PR @SHARANcanCODE! |
@@ -85,6 +85,35 @@ designed to help produce and consume Arrow data. | |||
- `nanoarrow Docs <https://arrow.apache.org/nanoarrow>`_ :fa:`external-link-alt` | |||
- `nanoarrow Source <http://github.com/apache/arrow-nanoarrow>`_ | |||
|
|||
Third-party Implementations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think as per comments in the issue, nanoarrow can be moved under the list for Official Implementations, cc @paleolimbot (in case you might suggest some changes in the text)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to do it here, but maybe above:
* - C (GLib)
- :doc:`C GLib Docs<c_glib/index>`
* - C (Native) (or: Minimal?)
- `nanoarrow Docs <https://arrow.apache.org/nanoarrow>`_ :fa:`external-link-alt`
- `nanoarrow Source <http://github.com/apache/arrow-nanoarrow>`_
...and removing the nanoarrow-specific section?
* - Sparrow | ||
- C++ | ||
- C++20 implementation of Apache Arrow | ||
- `Sparrow Docs <https://github.com/man-group/sparrow>`_ :fa:`external-link-alt` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `Sparrow Docs <https://github.com/man-group/sparrow>`_ :fa:`external-link-alt` | |
- `Sparrow Docs <https://man-group.github.io/sparrow/index.html>`_ :fa:`external-link-alt` |
- Python library for Apache Arrow using WebAssembly | ||
- `arro3 Docs <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` | ||
- `arro3 Source <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Python library for Apache Arrow using WebAssembly | |
- `arro3 Docs <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` | |
- `arro3 Source <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` | |
- Python library for Apache Arrow using arrow-rs | |
- `arro3 Docs <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` | |
- `arro3 Source <https://github.com/kylebarron/arro3>`_ :fa:`external-link-alt` |
@kylebarron Any suggestions on the wording here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest making the docs link point to https://kylebarron.dev/arro3/latest/, not github.
I'd probably also spell out as below. Not everyone might understand that arrow-rs
is the name of the official rust implementation?
Python library for Apache Arrow binding to the [Arrow Rust implementation](https://github.com/apache/arrow-rs)
@@ -85,6 +85,35 @@ designed to help produce and consume Arrow data. | |||
- `nanoarrow Docs <https://arrow.apache.org/nanoarrow>`_ :fa:`external-link-alt` | |||
- `nanoarrow Source <http://github.com/apache/arrow-nanoarrow>`_ | |||
|
|||
Third-party Implementations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to do it here, but maybe above:
* - C (GLib)
- :doc:`C GLib Docs<c_glib/index>`
* - C (Native) (or: Minimal?)
- `nanoarrow Docs <https://arrow.apache.org/nanoarrow>`_ :fa:`external-link-alt`
- `nanoarrow Source <http://github.com/apache/arrow-nanoarrow>`_
...and removing the nanoarrow-specific section?
Rationale for this change
This PR addresses issue #47472 by adding a new section to list third-party open source implementations of Apache Arrow.
What changes are included in this PR?
Are these changes tested?
The changes are documentation-only and follow the existing RST formatting patterns in the file.
Are there any user-facing changes?
Yes - users will now see a new section listing third-party implementations in the documentation.
Closes #47472