This is an automated email from the ASF dual-hosted git repository.
fokko pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg.git
The following commit(s) were added to refs/heads/main by this push:
new aeb26820a8 Hive: Remove deprecated `setSchema(TableMetadata,
Map<String, String>)` (#10257)
aeb26820a8 is described below
commit aeb26820a82859323c46f1879bd96f932a3cfca5
Author: Fokko Driesprong <[email protected]>
AuthorDate: Wed May 1 17:11:29 2024 +0200
Hive: Remove deprecated `setSchema(TableMetadata, Map<String, String>)`
(#10257)
---
.../src/main/java/org/apache/iceberg/hive/HiveOperationsBase.java | 8 --------
.../src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java | 2 +-
2 files changed, 1 insertion(+), 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
index a93577a35c..055a14246e 100644
---
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 @@ interface HiveOperationsBase {
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
index 369ad46c8e..95bf6c697c 100644
--- 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 class TestHiveCatalog extends
CatalogTests<HiveCatalog> {
.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);