ianmcook commented on a change in pull request #11156:
URL: https://github.com/apache/arrow/pull/11156#discussion_r708591935
##########
File path: r/R/dplyr-mutate.R
##########
@@ -38,11 +38,13 @@ mutate.arrow_dplyr_query <- function(.data,
.data <- as_adq(.data)
# Restrict the cases we support for now
- if (length(dplyr::group_vars(.data)) > 0) {
+ has_aggregations <- any(unlist(lapply(exprs, all_funs)) %in%
names(agg_funcs))
Review comment:
Fixed in 86ac1351069a7387342245e2230239f4182f2460 and added tests in
be7d9e04f3efd38b608ca88e14a1a412e1365cda that exercise the fix.
--
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]
