thisisnic commented on a change in pull request #10381:
URL: https://github.com/apache/arrow/pull/10381#discussion_r641398501
##########
File path: r/R/dplyr.R
##########
@@ -31,6 +31,18 @@ arrow_dplyr_query <- function(.data) {
return(.data)
}
+ # Evaluating expressions on a dataset with duplicated fieldnames will error
+ if (anyDuplicated(names(.data))) {
+ abort(c(
+ "Duplicated field names",
+ x = paste0(
+ "The following field names were found more than once in the data: ",
+ paste(names(.data)[duplicated(names(.data))], collapse = ", ")
+ ),
+ i = "All field names must be unique"
+ ))
+ }
Review comment:
Fair comment, it does not, and you're right it's redundant - have
removed it now.
--
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]
