Skip to content

Conversation

@Moohan
Copy link

@Moohan Moohan commented Jan 13, 2026

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.

@github-actions
Copy link

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?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@HyukjinKwon
Copy link
Member

Might need to add MINOR: at the PR title 👍

Copy link
Member

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants