[GitHub] [iceberg] RussellSpitzer commented on a change in pull request #2415: Spark: Add an action to remove all referenced files

[GitBox <gi...@apache.org>]

RussellSpitzer commented on a change in pull request #2415:
URL: https://github.com/apache/iceberg/pull/2415#discussion_r646808811



##########
File path: api/src/main/java/org/apache/iceberg/actions/ActionsProvider.java
##########
@@ -67,4 +67,12 @@ default RewriteDataFiles rewriteDataFiles(Table table) {
   default ExpireSnapshots expireSnapshots(Table table) {
     throw new UnsupportedOperationException(this.getClass().getName() + " does not implement expireSnapshots");
   }
+
+  /**
+   * Instantiates an action to remove all the files reachable from given metadata location.
+   */
+  default RemoveReachableFiles removeReachableFiles(String metadataLocation) {
+    throw new UnsupportedOperationException(this.getClass().getName() + " does not implement " +
+        RemoveReachableFiles.class.toString());

Review comment:
       nit: the others use the method name in the api and not the class name of the api




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org