thisisnic commented on code in PR #13786:
URL: https://github.com/apache/arrow/pull/13786#discussion_r952232030
##########
r/R/dplyr-mutate.R:
##########
@@ -24,7 +24,9 @@ mutate.arrow_dplyr_query <- function(.data,
.before = NULL,
.after = NULL) {
call <- match.call()
- exprs <- ensure_named_exprs(quos(...))
+
+ expression_list <- unfold_across(.data, quos(...))
Review Comment:
It's not as straightforward as I need to delve into the internals of
`do_arrow_summarize()` and swap some bits around. Given this PR is not tiny
and I'd be wanting to add a lot of tests for all 3, I'd like to keep them
separate to try to keep things simpler for review.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
