ajantha-bhat commented on code in PR #6091:
URL: https://github.com/apache/iceberg/pull/6091#discussion_r1121272260
##########
core/src/main/java/org/apache/iceberg/ReachableFileUtil.java:
##########
@@ -137,9 +137,23 @@ public static List<String> manifestListLocations(Table
table, Set<Long> snapshot
* @return the location of statistics files
*/
public static List<String> statisticsFilesLocations(Table table) {
+ return statisticsFilesLocations(table, null);
+ }
+
+ /**
+ * Returns locations of statistics files matching the given snapshotIds in a
table.
+ *
+ * @param table table for which statistics files needs to be listed
+ * @param snapshotIds ids of snapshots for which statistics files will be
returned. When null,
+ * returns location of all the statistics files in a table.
+ * @return the location of statistics files
+ */
+ public static List<String> statisticsFilesLocations(Table table, Set<Long>
snapshotIds) {
Review Comment:
Updated now.
Originally I wanted to keep the same style as `manifestListLocations` and
others. So, I didn't used modified.
--
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]
