fix

yuzelin · yuzelin · commit 47931ad56969 · 2024-11-12T09:44:30.000+08:00
diff --git a/paimon_python_java/paimon-python-java-bridge/src/main/java/org/apache/paimon/python/BytesWriter.java b/paimon_python_java/paimon-python-java-bridge/src/main/java/org/apache/paimon/python/BytesWriter.java
@@ -87,7 +87,7 @@ private boolean checkSchema(List<Field> expectedFields, List<Field> actualFields
         for (int i = 0; i < expectedFields.size(); i++) {
             Field expectedField = expectedFields.get(i);
             Field actualField = actualFields.get(i);
-            if (!checkField(expectedField, actualField)
+            if (!checkFieldIgnoreNullability(expectedField, actualField)
                     || !checkSchema(expectedField.getChildren(), actualField.getChildren())) {
                 return false;
             }
@@ -96,7 +96,7 @@ private boolean checkSchema(List<Field> expectedFields, List<Field> actualFields
         return true;
     }
 
-    private boolean checkField(Field expected, Field actual) {
+    private boolean checkFieldIgnoreNullability(Field expected, Field actual) {
         return Objects.equals(expected.getName(), actual.getName())
                 && Objects.equals(expected.getType(), actual.getType());
     }

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ private boolean checkSchema(List<Field> expectedFields, List<Field> actualFields`
`87`	`87`	`for (int i = 0; i < expectedFields.size(); i++) {`
`88`	`88`	`Field expectedField = expectedFields.get(i);`
`89`	`89`	`Field actualField = actualFields.get(i);`
`90`		`- if (!checkField(expectedField, actualField)`
	`90`	`+ if (!checkFieldIgnoreNullability(expectedField, actualField)`
`91`	`91`	`\|\| !checkSchema(expectedField.getChildren(), actualField.getChildren())) {`
`92`	`92`	`return false;`
`93`	`93`	`}`
`@@ -96,7 +96,7 @@ private boolean checkSchema(List<Field> expectedFields, List<Field> actualFields`
`96`	`96`	`return true;`
`97`	`97`	`}`
`98`	`98`
`99`		`- private boolean checkField(Field expected, Field actual) {`
	`99`	`+ private boolean checkFieldIgnoreNullability(Field expected, Field actual) {`
`100`	`100`	`return Objects.equals(expected.getName(), actual.getName())`
`101`	`101`	`&& Objects.equals(expected.getType(), actual.getType());`
`102`	`102`	`}`