[GitHub] [arrow] kszucs commented on pull request #7073: ARROW-8318: [C++][Dataset] Construct FileSystemDataset from fragments

2020-05-01 Thread GitBox
kszucs commented on pull request #7073: URL: https://github.com/apache/arrow/pull/7073#issuecomment-622410968 I'd like elaborate a bit more on the generic dataset class regardless what kind of wrappers do we provide. - Do you plan to unify the filesystem classes into a single one which

[GitHub] [arrow] kszucs commented on pull request #7073: ARROW-8318: [C++][Dataset] Construct FileSystemDataset from fragments

2020-05-01 Thread GitBox
kszucs commented on pull request #7073: URL: https://github.com/apache/arrow/pull/7073#issuecomment-62291 > * Simplified FileSystemDataset to hold a FragmentVector. Each Fragment must be a FileFragment and is checked at `FileSystemDataset::Make`. Fragments are not required to use the