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



##########
File path: r/NEWS.md
##########
@@ -21,7 +21,7 @@
 
 ## dplyr methods
 
-* `dplyr::mutate()` on Arrow `Table` and `RecordBatch` is now supported in 
Arrow for many applications. Where not yet supported, the implementation falls 
back to pulling data into an R `data.frame` first.
+* `dplyr::mutate()` is now supported in Arrow for many applications. For 
queries on `Table` and `RecordBatch` that are not yet supported in Arrow, the 
implementation falls back to pulling data into an R `data.frame` first for, as 
in the previous release. For queries on `Dataset`, it raises an error if the 
feature is not implemented.

Review comment:
       ```suggestion
   * `dplyr::mutate()` is now supported in Arrow for many applications. For 
queries on `Table` and `RecordBatch` that are not yet supported in Arrow, the 
implementation falls back to pulling data into an R `data.frame` first, as in 
the previous release. For queries on `Dataset`, it raises an error if the 
feature is not implemented.
   ```




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to