compare argument type in a null-friendly way (#38092)

ahmedabu98 · web-flow · commit 8e40d4450b7c · 2026-04-07T15:03:21.000-07:00
diff --git a/.github/trigger_files/beam_PostCommit_SQL.json b/.github/trigger_files/beam_PostCommit_SQL.json
@@ -1,4 +1,4 @@
 {
   "comment": "Modify this file in a trivial way to cause this test suite to run ",
-  "modification": 0
+  "modification": 1
 }
diff --git a/sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/Schema.java b/sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/Schema.java
@@ -974,7 +974,8 @@ public boolean typesEqual(FieldType other) {
             getLogicalType().getIdentifier(), other.getLogicalType().getIdentifier())) {
           return false;
         }
-        if (!getLogicalType().getArgumentType().equals(other.getLogicalType().getArgumentType())) {
+        if (!Objects.equals(
+            getLogicalType().getArgumentType(), other.getLogicalType().getArgumentType())) {
           return false;
         }
         if (!Row.Equals.deepEquals(

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`	`1`	`{`
`2`	`2`	`"comment": "Modify this file in a trivial way to cause this test suite to run ",`
`3`		`- "modification": 0`
	`3`	`+ "modification": 1`
`4`	`4`	`}`
Original file line number	Diff line number	Diff line change
`@@ -974,7 +974,8 @@ public boolean typesEqual(FieldType other) {`
`974`	`974`	`getLogicalType().getIdentifier(), other.getLogicalType().getIdentifier())) {`
`975`	`975`	`return false;`
`976`	`976`	`}`
`977`		`- if (!getLogicalType().getArgumentType().equals(other.getLogicalType().getArgumentType())) {`
	`977`	`+ if (!Objects.equals(`
	`978`	`+ getLogicalType().getArgumentType(), other.getLogicalType().getArgumentType())) {`
`978`	`979`	`return false;`
`979`	`980`	`}`
`980`	`981`	`if (!Row.Equals.deepEquals(`