pvary commented on code in PR #13302:
URL: https://github.com/apache/iceberg/pull/13302#discussion_r2166767716
##########
core/src/main/java/org/apache/iceberg/util/FileSystemWalker.java:
##########
@@ -40,6 +43,26 @@ public class FileSystemWalker {
private FileSystemWalker() {}
+ /**
+ * Recursively lists files in the specified directory that satisfy the given
conditions. Use
+ * {@link PartitionAwareHiddenPathFilter} to filter out hidden paths.
+ *
+ * @param io File system interface supporting prefix operations
+ * @param dir Base directory to start recursive listing
+ * @param specs Map of {@link PartitionSpec partition specs} for this table.
+ * @param predicate Additional filter condition for files
+ * @param consumer Consumer to accept matching file locations
+ */
+ public static void listDirRecursivelyWithFileIO(
Review Comment:
Maybe we can remove the other public methods
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
