ianmcook commented on a change in pull request #9800:
URL: https://github.com/apache/arrow/pull/9800#discussion_r607390178
##########
File path: r/R/dataset.R
##########
@@ -64,6 +73,8 @@ open_dataset <- function(sources,
partitioning = hive_partition(),
unify_schemas = NULL,
...) {
+ # TODO: default to unify_schemas = TRUE when `sources` is a vector of file
paths/URIs?
Review comment:
The docs say
>The default is `FALSE` when creating a dataset from a file path (because
there may be many files and scanning may be slow)
If the user is specifying the paths to the files, it seems likely that there
would be fewer files, so maybe it would make sense to have a different default?
If you don't have any strong opinions, then we can leave it as is.
--
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
