[ARROW-18203] [R] Refactor to remove unnecessary uses of build_expr

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 11.0.0
• Component/s: R
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/33389
• Assignee: Neal Richardson
• Reporter: Neal Richardson
• Created: 31/Oct/22 15:23
• Updated: 11/Jan/23 11:58
• Resolved: 02/Nov/22 21:56

Description

In ARROW-17462 we added some logic to build_expr() to cast scalars in expressions to match the type of the field. It was observed in review that this was not something you'd want everywhere and you should have to opt-in to it. We should refactor our internals so that we only use build_expr() where we want this behavior, and otherwise use Expression$create() directly.

Issue Links

• links to: GitHub Pull Request #14553