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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
