Skip to content

Commit

Permalink
GH-44069: [Docs][R] Add note to to_arrow() docs about collect/compute (
Browse files Browse the repository at this point in the history
…#44094)

### Rationale for this change

Improves the documentation for the `to_arrow()` function for the use case referenced in #44069.

### What changes are included in this PR?

Just docs.

### Are these changes tested?

Yes. Built and tested locally.

### Are there any user-facing changes?

Just docs.
* GitHub Issue: #44069

Authored-by: Bryce Mecum <[email protected]>
Signed-off-by: Nic Crane <[email protected]>
  • Loading branch information
amoeba authored Sep 14, 2024
1 parent 3eb4135 commit 41c481f
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 2 deletions.
8 changes: 7 additions & 1 deletion r/R/duckdb.R
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,13 @@ duckdb_disconnector <- function(con, tbl_name) {

#' Create an Arrow object from a DuckDB connection
#'
#' This can be used in pipelines that pass data back and forth between Arrow and DuckDB
#' This can be used in pipelines that pass data back and forth between Arrow and
#' DuckDB.
#'
#' Note that you can only call `collect()` or `compute()` on the result of this
#' function once. To work around this limitation, you should either only call
#' `collect()` as the final step in a pipeline or call `as_arrow_table()` on the
#' result to materialize the entire Table in-memory.
#'
#' @param .data the object to be converted
#' @return A `RecordBatchReader`.
Expand Down
9 changes: 8 additions & 1 deletion r/man/to_arrow.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 41c481f

Please sign in to comment.