[GitHub] [arrow] ianmcook commented on a change in pull request #10672: ARROW-13262: [R] transmute() fails after pulling data into R

[GitBox <gi...@apache.org>]

ianmcook commented on a change in pull request #10672:
URL: https://github.com/apache/arrow/pull/10672#discussion_r665415888



##########
File path: r/NEWS.md
##########
@@ -24,7 +24,8 @@
 * `arrow_info()` now includes details on the C++ build, such as compiler version
 * `dplyr` queries on `Table` and `RecordBatch` now use the same expression internals as `Dataset` (via `InMemoryDataset`). Among other (mostly internal) benefits that come with this, the print method for `arrow_dplyr_query` now includes the expression and the resulting type of columns derived by `mutate()`.
 * Added bindings for the remainder of C data interface: Type, Field, and RecordBatchReader (from the experimental C stream interface). These also have `reticulate::py_to_r()` and `r_to_py()` methods. Along with the addition of the `Scanner$ToRecordBatchReader()` method, you can now build up a Dataset query in R and pass the resulting stream of batches to another tool in process.
-* `match_arrow` now converts `x` into an `Array` if it is not a `Scalar`, `Array` or `ChunkedArray` and no longer dispatches `base::match()`.
+* `match_arrow()` now converts `x` into an `Array` if it is not a `Scalar`, `Array` or `ChunkedArray` and no longer dispatches `base::match()`.
+* `transmute()` now errors if passed arguments `.keep`, `.before`, or `.after`, for consistency with the behavior of `dplyr` on `data.frame`s.

Review comment:
       Since this is a potentially breaking change for folks who are (ab)using `.keep`, `.before`, or `.after`, I added a NEWS item about it.




-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org