Details
-
Improvement
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
Description
Originally raised as part of PR #12433.
This implementation made me think of the various as.* methods we have defined [1] (since this is similar to as.Date()). Which all use a simpler setup to create a cast operation. However, I noticed that for those, they are using Expression$create(...) rather than the build_expr(...) helper [2]. That build_expr(...) here should handle the wrapping of R objects into Scalars (...)
(...)We should also open a jira if one doesn't exist to clean up those as.* methods to use build_expr()
Attachments
Issue Links
- links to