nealrichardson commented on a change in pull request #10955:
URL: https://github.com/apache/arrow/pull/10955#discussion_r709212523



##########
File path: r/tests/testthat/test-metadata.R
##########
@@ -231,6 +231,9 @@ test_that("metadata of list elements (ARROW-10386)", {
     "Row-level metadata is not compatible with datasets and will be discarded"
   )
 
+  # Reset directory as previous write will have created some files and the 
defualt

Review comment:
       So this is configurable and we've changed the default? Can we set the 
default in the R bindings to be the status quo, just to eliminate surprises?
   
   Also:
   ```suggestion
     # Reset directory as previous write will have created some files and the 
default
   ```




-- 
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-unsubscr...@arrow.apache.org

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


Reply via email to