Remove setting metastore SerDe name

Author: jinyangli34

plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/thrift/ThriftMetastoreUtil.java

@@ -192,7 +192,7 @@ public static io.trino.hive.thrift.metastore.Table toMetastoreApiTable(Table tab
         result.setTableType(table.getTableType());
         result.setParameters(table.getParameters());
         result.setPartitionKeys(table.getPartitionColumns().stream().map(ThriftMetastoreUtil::toMetastoreApiFieldSchema).collect(toImmutableList()));
-        result.setSd(makeStorageDescriptor(table.getTableName(), table.getDataColumns(), table.getStorage()));
+        result.setSd(makeStorageDescriptor(table.getDataColumns(), table.getStorage()));
         result.setViewOriginalText(table.getViewOriginalText().orElse(null));
         result.setViewExpandedText(table.getViewExpandedText().orElse(null));
         table.getWriteId().ifPresent(result::setWriteId);
@@ -350,7 +350,7 @@ public static io.trino.hive.thrift.metastore.Partition toMetastoreApiPartition(P
         result.setDbName(partition.getDatabaseName());
         result.setTableName(partition.getTableName());
         result.setValues(partition.getValues());
-        result.setSd(makeStorageDescriptor(partition.getTableName(), partition.getColumns(), partition.getStorage()));
+        result.setSd(makeStorageDescriptor(partition.getColumns(), partition.getStorage()));
         result.setParameters(partition.getParameters());
         writeId.ifPresent(result::setWriteId);
         return result;
@@ -679,10 +679,9 @@ private static void fromMetastoreApiStorageDescriptor(
                 .setSerdeParameters(serdeInfo.getParameters() == null ? ImmutableMap.of() : serdeInfo.getParameters());
     }
 
-    private static StorageDescriptor makeStorageDescriptor(String tableName, List<Column> columns, Storage storage)
+    private static StorageDescriptor makeStorageDescriptor(List<Column> columns, Storage storage)
     {
         SerDeInfo serdeInfo = new SerDeInfo();
-        serdeInfo.setName(tableName);
         serdeInfo.setSerializationLib(storage.getStorageFormat().getSerDeNullable());
         serdeInfo.setParameters(storage.getSerdeParameters());
 

plugin/trino-hive/src/test/java/io/trino/plugin/hive/metastore/thrift/TestThriftMetastoreUtil.java

@@ -97,7 +97,7 @@ public class TestThriftMetastoreUtil
             "com.facebook.hive.orc.OrcOutputFormat",
             false,
             100,
-            new SerDeInfo("table_name", "com.facebook.hive.orc.OrcSerde", ImmutableMap.of("sdk1", "sdv1", "sdk2", "sdv2")),
+            new SerDeInfo(null, "com.facebook.hive.orc.OrcSerde", ImmutableMap.of("sdk1", "sdv1", "sdk2", "sdv2")),
             ImmutableList.of("col2", "col3"),
             ImmutableList.of(new Order("col2", 1)),
             ImmutableMap.of());
@@ -136,7 +136,7 @@ public class TestThriftMetastoreUtil
             "com.facebook.hive.orc.OrcOutputFormat",
             false,
             100,
-            new SerDeInfo("table_name", "com.facebook.hive.orc.OrcSerde", ImmutableMap.of("sdk1", "sdv1", "sdk2", "sdv2")),
+            new SerDeInfo(null, "com.facebook.hive.orc.OrcSerde", ImmutableMap.of("sdk1", "sdv1", "sdk2", "sdv2")),
             ImmutableList.of("col2", "col3"),
             ImmutableList.of(new Order("col2", 0), new Order("col3", 1)),
             ImmutableMap.of("sk1", "sv1"));