modify the CalcitePPLFunctionTypeTest text

Signed-off-by: Xinyu Hao <[email protected]>

Author: ishaoxy

core/src/main/java/org/opensearch/sql/expression/function/PPLTypeChecker.java

@@ -400,6 +400,10 @@ private static List<ExprType> getExprTypes(SqlTypeFamily family) {
               OpenSearchTypeFactory.TYPE_FACTORY.createSqlType(SqlTypeName.INTEGER));
           case ANY, IGNORE -> List.of(
               OpenSearchTypeFactory.TYPE_FACTORY.createSqlType(SqlTypeName.ANY));
+            // We borrow SqlTypeFamily.NULL to represent EXPR_IP. This is a workaround
+            // since there is no corresponding IP type family in Calcite.
+          case NULL -> List.of(
+              OpenSearchTypeFactory.TYPE_FACTORY.createUDT(OpenSearchTypeFactory.ExprUDT.EXPR_IP));
           default -> {
             RelDataType type = family.getDefaultConcreteType(OpenSearchTypeFactory.TYPE_FACTORY);
             if (type == null) {

ppl/src/test/java/org/opensearch/sql/ppl/calcite/CalcitePPLFunctionTypeTest.java

@@ -49,7 +49,9 @@ public void testComparisonWithDifferentType() {
     String ppl = "source=EMP | where ENAME < 6 | fields ENAME";
     Throwable t = Assert.assertThrows(ExpressionEvaluationException.class, () -> getRelNode(ppl));
     verifyErrorMessageContains(
-        t, "LESS function expects {[COMPARABLE_TYPE,COMPARABLE_TYPE]}, but got [STRING,INTEGER]");
+        t,
+        "LESS function expects {[STRING,IP],[IP,STRING],[IP,IP],[COMPARABLE_TYPE,COMPARABLE_TYPE]},"
+            + " but got [STRING,INTEGER]");
   }
 
   @Test