RussellSpitzer commented on a change in pull request #4397:
URL: https://github.com/apache/iceberg/pull/4397#discussion_r836580573
##########
File path: data/src/main/java/org/apache/iceberg/data/TableMigrationUtil.java
##########
@@ -133,6 +132,20 @@ private TableMigrationUtil() {
}
}
+ private static List<FileStatus> listStatus(Configuration conf, String
partitionUri) throws IOException {
Review comment:
@kbendick Iceberg doesn't care where files are so as long as all the
files belong to the correct partition in the sub directories I have no issue
with it. A file's path should have no effect on any other iceberg operations.
This could be tricky for users if they somehow kept their partitions in some
odd nested pattern but Ideally i think we should support listing files in the
same way Spark does.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
