Fix failing test case

suddendust · suddendust · commit b59956462f8a · 2025-12-18T17:00:20.000+05:30
diff --git a/document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/nonjson/field/PostgresInRelationalFilterParserArrayField.java b/document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/nonjson/field/PostgresInRelationalFilterParserArrayField.java
@@ -97,8 +97,9 @@ private String prepareFilterStringForArrayInOperator(
       paramsBuilder.addArrayParam(values, sqlType);
       return String.format("%s && ?", parsedLhs);
     } else {
-      // Fallback: use jsonb array contains approach
-      return prepareFilterStringFallback(parsedLhs, parsedRhs, paramsBuilder, "%s @> ARRAY[%s]");
+      // Fallback: cast both sides to text[] for backward compatibility with any array type
+      return prepareFilterStringFallback(
+          parsedLhs, parsedRhs, paramsBuilder, "%s::text[] && ARRAY[%s]::text[]");
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -97,8 +97,9 @@ private String prepareFilterStringForArrayInOperator(`
`97`	`97`	`paramsBuilder.addArrayParam(values, sqlType);`
`98`	`98`	`return String.format("%s && ?", parsedLhs);`
`99`	`99`	`} else {`
`100`		`- // Fallback: use jsonb array contains approach`
`101`		`- return prepareFilterStringFallback(parsedLhs, parsedRhs, paramsBuilder, "%s @> ARRAY[%s]");`
	`100`	`+ // Fallback: cast both sides to text[] for backward compatibility with any array type`
	`101`	`+ return prepareFilterStringFallback(`
	`102`	`+ parsedLhs, parsedRhs, paramsBuilder, "%s::text[] && ARRAY[%s]::text[]");`
`102`	`103`	`}`
`103`	`104`	`}`
`104`	`105`