bkietz commented on a change in pull request #10191:
URL: https://github.com/apache/arrow/pull/10191#discussion_r625802268
##########
File path: r/tests/testthat/test-dplyr-mutate.R
##########
@@ -344,20 +344,21 @@ test_that("print a mutated table", {
select(int) %>%
mutate(twice = int * 2) %>%
print(),
-'Table (query)
+'InMemoryDataset (query)
int: int32
twice: expr
See $.data for the source Arrow object',
fixed = TRUE)
# Handling non-expressions/edge cases
+ skip("InMemoryDataset$Project() doesn't accept array (or could it?)")
expect_output(
Table$create(tbl) %>%
select(int) %>%
mutate(again = 1:10) %>%
Review comment:
The workaround I proposed and detection of an in memory scan would occur
in R. If we don't *need* to support mutate with vectors then I agree that it'd
be ideal to avoid dealing with it and just document that it will not be
supported
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
