Parameter order change

Author: pvary

spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/data/SparkParquetWriters.java

@@ -78,19 +78,28 @@ public class SparkParquetWriters {
   private SparkParquetWriters() {}
 
   public static <T> ParquetValueWriter<T> buildWriter(StructType dfSchema, MessageType type) {
-    return buildWriter(null, dfSchema, type);
+    return buildWriter(null, type, dfSchema);
   }
 
   @SuppressWarnings("unchecked")
   public static <T> ParquetValueWriter<T> buildWriter(
-      Schema icebergSchema, StructType dfSchema, MessageType type) {
+      Schema icebergSchema, MessageType type, StructType dfSchema) {
     return (ParquetValueWriter<T>)
         ParquetWithSparkSchemaVisitor.visit(
             dfSchema != null ? dfSchema : SparkSchemaUtil.convert(icebergSchema),
             type,
             new WriteBuilder(type));
   }
 
+    public static <T> ParquetValueWriter<T> buildWriter(
+            StructType dfSchema, MessageType type, Schema icebergSchema) {
+        return (ParquetValueWriter<T>)
+                ParquetWithSparkSchemaVisitor.visit(
+                        dfSchema != null ? dfSchema : SparkSchemaUtil.convert(icebergSchema),
+                        type,
+                        new WriteBuilder(type));
+    }
+
   private static class WriteBuilder extends ParquetWithSparkSchemaVisitor<ParquetValueWriter<?>> {
     private final MessageType type;
 

spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/source/SparkFormatModels.java

@@ -48,8 +48,7 @@ public static void register() {
         ParquetFormatModel.create(
             InternalRow.class,
             StructType.class,
-            (icebergSchema, fileSchema, engineSchema) ->
-                SparkParquetWriters.buildWriter(icebergSchema, engineSchema, fileSchema),
+            SparkParquetWriters::buildWriter,
             (icebergSchema, fileSchema, engineSchema, idToConstant) ->
                 SparkParquetReaders.buildReader(icebergSchema, fileSchema, idToConstant)));