Recorder hive table properties to align the avro.schema.literal placement contract (linkedin#86)

rzhang10 · rzhang10 · commit 987eb1df7d81 · 2022-09-28T10:15:37.000-07:00
diff --git a/hive-metastore/src/main/java/org/apache/iceberg/hive/legacy/LegacyHiveTableUtils.java b/hive-metastore/src/main/java/org/apache/iceberg/hive/legacy/LegacyHiveTableUtils.java
@@ -126,9 +126,9 @@ private static Type primitiveIcebergType(String hiveTypeString) {
 
   static Map<String, String> getTableProperties(org.apache.hadoop.hive.metastore.api.Table table) {
     Map<String, String> props = new HashMap<>();
-    props.putAll(table.getSd().getSerdeInfo().getParameters());
     props.putAll(table.getSd().getParameters());
     props.putAll(table.getParameters());
+    props.putAll(table.getSd().getSerdeInfo().getParameters());
     return props;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -126,9 +126,9 @@ private static Type primitiveIcebergType(String hiveTypeString) {`
`126`	`126`
`127`	`127`	`static Map<String, String> getTableProperties(org.apache.hadoop.hive.metastore.api.Table table) {`
`128`	`128`	`Map<String, String> props = new HashMap<>();`
`129`		`- props.putAll(table.getSd().getSerdeInfo().getParameters());`
`130`	`129`	`props.putAll(table.getSd().getParameters());`
`131`	`130`	`props.putAll(table.getParameters());`
	`131`	`+ props.putAll(table.getSd().getSerdeInfo().getParameters());`
`132`	`132`	`return props;`
`133`	`133`	`}`
`134`	`134`