Conversation
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or See also: |
|
Might need to add |
jonkeane
left a comment
There was a problem hiding this comment.
Thanks for reaching out and the proactive contribution. Though I think this is not actually a typo. I am inclined to close this, but maybe there is more detail we could add that explains what's going on here. Thoughts?
| #' # Function mappings | ||
| #' | ||
| #' In the list below, any differences in behavior or support between Acero and | ||
| #' In the list below, any differences in behavior or support between Arrow and |
There was a problem hiding this comment.
This isn't actually a typo. Acero is the name of the streaming C++ query engine that the Arrow R package uses. See https://arrow.apache.org/docs/cpp/acero.html
We do mention this for example here: https://arrow.apache.org/docs/r/reference/acero.html?q=acero#null which is also above this at https://github.com/apache/arrow/pull/48840/changes#diff-8e2b0f2944713c80b312477fd9e90bf65071251dd380b84940127cc167c7b55aR27-R28
Rationale for this change
Noticed a typo when reading the docs
What changes are included in this PR?
A single typo fix
Are these changes tested?
n/a
Are there any user-facing changes?
Yes - Docs only.