ajantha-bhat commented on code in PR #8502: URL: https://github.com/apache/iceberg/pull/8502#discussion_r1422541957
########## core/src/main/java/org/apache/iceberg/ReachableFileUtil.java: ########## @@ -137,7 +137,9 @@ public static List<String> manifestListLocations(Table table, Set<Long> snapshot * * @param table table for which statistics files needs to be listed * @return the location of statistics files + * @deprecated use the {@code allStatisticsFilesLocations(table)} instead. */ + @Deprecated Review Comment: removed the changes in this file as the new APIs added was not called from anywhere (this is needed for spark action). I will raise a follow up PR for Spark actions to consider partition stats file for remove orphan files and expire snapshots action. -- 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