(arrow) branch main updated: GH-40199: [R] dbplyr 2.5.0 forward compatibility (#40197)

[pa...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 65c2b46c83 GH-40199: [R] dbplyr 2.5.0 forward compatibility (#40197)
65c2b46c83 is described below

commit 65c2b46c83da0f82f4cf6531cde938edb0327f49
Author: Hadley Wickham <h....@gmail.com>
AuthorDate: Fri Feb 23 09:36:45 2024 -0600

    GH-40199: [R] dbplyr 2.5.0 forward compatibility (#40197)
    
    I'm not sure why this worked before, but 2.5.0 got a little stricter and now warns about this minor infelicity.
    
    I _think_ this is the only issue with 2.5.0, but if there's any way you could run your CI checks with dev dbplyr I'd really appreciate it!
    
    * GitHub Issue: #40199
    
    Authored-by: Hadley Wickham <h....@gmail.com>
    Signed-off-by: Dewey Dunnington <de...@voltrondata.com>
---
 r/R/duckdb.R | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/r/R/duckdb.R b/r/R/duckdb.R
index 9632e9bad1..a2bf62de2f 100644
--- a/r/R/duckdb.R
+++ b/r/R/duckdb.R
@@ -64,7 +64,7 @@ to_duckdb <- function(.data,
   tbl <- dplyr::tbl(con, table_name)
   groups <- dplyr::groups(.data)
   if (length(groups)) {
-    tbl <- dplyr::group_by(tbl, groups)
+    tbl <- dplyr::group_by(tbl, !!!groups)
   }
 
   if (auto_disconnect) {