[ARROW-13117] [R] Retain schema in new Expressions

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.0.0
• Component/s: R
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/28820
• Assignee: Ian Cook
• Reporter: Ian Cook
• Created: 18/Jun/21 14:58
• Updated: 11/Jan/23 08:31
• Resolved: 25/Jun/21 20:57

Description

When a new Expression is created, schema should be retained from the expression(s) it was created from. That way, the type() and type_id() methods of the new Expression will work. For example, currently this happens:

> x <- Expression$field_ref("x")
> x$schema <- Schema$create(x = int32())
> 
> y <- Expression$field_ref("y")
> y$schema <- Schema$create(y = int32())
> 
> Expression$create("add_checked", x, y)$type()
Error: !is.null(schema) is not TRUE 

This is what we want to happen:

> Expression$create("add_checked", x, y)$type()
Int32
int32

Issue Links

• is related to: ARROW-13119 [R] Set empty schema in scalar Expressions
• links to: GitHub Pull Request #10563