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