Skip to content

Commit

Permalink
MINOR: [R] fix no visible global function definition: left_join (apac…
Browse files Browse the repository at this point in the history
…he#41542)

### Rationale for this change

Followup to apache#41350, fixes a check NOTE that caused.

### What changes are included in this PR?

`dplyr::` in two places.

### Are these changes tested?

Check will be clean.

### Are there any user-facing changes?

🙅
  • Loading branch information
nealrichardson authored May 6, 2024
1 parent 4cf44b4 commit d10ebf0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions r/R/dplyr-mutate.R
Original file line number Diff line number Diff line change
Expand Up @@ -84,12 +84,12 @@ mutate.arrow_dplyr_query <- function(.data,
agg_query$aggregations <- mask$.aggregations
agg_query <- collapse.arrow_dplyr_query(agg_query)
if (length(grv)) {
out <- left_join(out, agg_query, by = grv)
out <- dplyr::left_join(out, agg_query, by = grv)
} else {
# If there are no group_by vars, add a scalar column to both and join on that
agg_query$selected_columns[["..tempjoin"]] <- Expression$scalar(1L)
out$selected_columns[["..tempjoin"]] <- Expression$scalar(1L)
out <- left_join(out, agg_query, by = "..tempjoin")
out <- dplyr::left_join(out, agg_query, by = "..tempjoin")
}
}

Expand Down

0 comments on commit d10ebf0

Please sign in to comment.