Hive: Remove deprecated `setSchema(TableMetadata, Map<String, String>…

…)` (#10257)
apache · May 1, 2024 · aeb2682 · aeb2682
1 parent 1757577
commit aeb2682
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveOperationsBase.java b/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveOperationsBase.java
@@ -103,14 +103,6 @@ default boolean exposeInHmsProperties() {
     return maxHiveTablePropertySize() > 0;
   }
 
-  /**
-   * @deprecated since 1.6.0, will be removed in 1.7.0; Use {@link #setSchema(Schema, Map)} instead
-   */
-  @Deprecated
-  default void setSchema(TableMetadata metadata, Map<String, String> parameters) {
-    setSchema(metadata.schema(), parameters);
-  }
-
   default void setSchema(Schema schema, Map<String, String> parameters) {
     parameters.remove(TableProperties.CURRENT_SCHEMA);
     if (exposeInHmsProperties() && schema != null) {

diff --git a/hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java b/hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java
@@ -1041,7 +1041,7 @@ public void testNotExposeTableProperties() {
         .doesNotContainKey(CURRENT_SNAPSHOT_ID)
         .doesNotContainKey(CURRENT_SNAPSHOT_TIMESTAMP);
 
-    ops.setSchema(metadata, parameters);
+    ops.setSchema(metadata.schema(), parameters);
     assertThat(parameters).doesNotContainKey(CURRENT_SCHEMA);
 
     ops.setPartitionSpec(metadata, parameters);