address comments

apache · Aug 1, 2024 · cd8cf08 · cd8cf08
1 parent d4c3956
commit cd8cf08
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/core/src/main/java/org/apache/iceberg/BaseTransaction.java b/core/src/main/java/org/apache/iceberg/BaseTransaction.java
@@ -690,7 +690,7 @@ public UpdateProperties updateProperties() {
     @Override
     public MetadataMaintenance maintenance() {
       throw new UnsupportedOperationException(
-          "Cannot maintenance metadata as part of a transaction");
+          "Cannot perform metadata maintenance as part of a transaction");
     }
 
     @Override

diff --git a/core/src/main/java/org/apache/iceberg/MetadataUpdate.java b/core/src/main/java/org/apache/iceberg/MetadataUpdate.java
@@ -175,13 +175,6 @@ public RemoveUnusedSpecs(Set<Integer> specIds) {
     public Set<Integer> specIds() {
       return specIds;
     }
-
-    @Override
-    public void applyTo(TableMetadata.Builder metadataBuilder) {
-      // metadataBuilder.removeUnusedSpecsById(specIds);
-      throw new UnsupportedOperationException(
-          "RemoveUnusedSpecs is not supported for REST catalog yet");
-    }
   }
 
   class AddSortOrder implements MetadataUpdate {