Use ImmutableList.Builder in CheckpointSchemaManager

ebyhr · ebyhr · commit 379154bbda40 · 2022-09-06T10:36:02.000+09:00
diff --git a/plugin/trino-delta-lake/src/main/java/io/trino/plugin/deltalake/transactionlog/checkpoint/CheckpointSchemaManager.java b/plugin/trino-delta-lake/src/main/java/io/trino/plugin/deltalake/transactionlog/checkpoint/CheckpointSchemaManager.java
@@ -137,17 +137,17 @@ public RowType getAddEntryType(MetadataEntry metadataEntry)
                 RowType.from(allColumns.stream().map(column -> buildNullCountType(Optional.of(column.getPhysicalName()), column.getPhysicalColumnType())).collect(toImmutableList()))));
 
         MapType stringMap = (MapType) typeManager.getType(TypeSignature.mapType(VarcharType.VARCHAR.getTypeSignature(), VarcharType.VARCHAR.getTypeSignature()));
-        List<RowType.Field> addFields = ImmutableList.of(
-                RowType.field("path", VarcharType.createUnboundedVarcharType()),
-                RowType.field("partitionValues", stringMap),
-                RowType.field("size", BigintType.BIGINT),
-                RowType.field("modificationTime", BigintType.BIGINT),
-                RowType.field("dataChange", BooleanType.BOOLEAN),
-                RowType.field("stats", VarcharType.createUnboundedVarcharType()),
-                RowType.field("stats_parsed", RowType.from(statsColumns.build())),
-                RowType.field("tags", stringMap));
-
-        return RowType.from(addFields);
+        ImmutableList.Builder<RowType.Field> addFields = ImmutableList.builder();
+        addFields.add(RowType.field("path", VarcharType.createUnboundedVarcharType()));
+        addFields.add(RowType.field("partitionValues", stringMap));
+        addFields.add(RowType.field("size", BigintType.BIGINT));
+        addFields.add(RowType.field("modificationTime", BigintType.BIGINT));
+        addFields.add(RowType.field("dataChange", BooleanType.BOOLEAN));
+        addFields.add(RowType.field("stats", VarcharType.createUnboundedVarcharType()));
+        addFields.add(RowType.field("stats_parsed", RowType.from(statsColumns.build())));
+        addFields.add(RowType.field("tags", stringMap));
+
+        return RowType.from(addFields.build());
     }
 
     private static RowType.Field buildNullCountType(Optional<String> columnName, Type columnType)
