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

Reply via email to