wjones127 commented on code in PR #12751:
URL: https://github.com/apache/arrow/pull/12751#discussion_r853142437
##########
r/R/record-batch.R:
##########
@@ -189,3 +189,59 @@ record_batch <- RecordBatch$create
#' @export
names.RecordBatch <- function(x) x$names()
+
+#' @export
+rbind.RecordBatch <- function(...) {
+ abort("Use `Table$create()` to combine record batches")
Review Comment:
This is a similar situation to `c.Array`, where there's a zero-copy solution
that gets you a chunked array and a solution that copies to give you an Array.
I was thinking that we'd rather the user made an explicit choice about what
they want, rather than make that for them. Though we don't have a
`concat_record_batches` function here yet, we would present that as another
option here if we did.
--
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]
