Rule out ip from built-in comparator via its type checker

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

Author: yuancu

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

@@ -86,15 +86,11 @@ public class CoercionUtils {
   }
 
   private static @Nullable RexNode cast(RexBuilder builder, ExprType targetType, RexNode arg) {
-    // Implement the logic to check if fromType can be cast to toType
-    // This could involve checking if the types are compatible, or if a cast is possible
-    // For example, you might check if fromType is a subtype of toType, or if a conversion exists
     ExprType argType = OpenSearchTypeFactory.convertRelDataTypeToExprType(arg.getType());
     if (!argType.shouldCast(targetType)) {
       return arg;
     }
 
-    // If the arg is string
     if (WideningTypeRule.distance(argType, targetType) != WideningTypeRule.IMPOSSIBLE_WIDENING) {
       return builder.makeCast(OpenSearchTypeFactory.convertExprTypeToRelDataType(targetType), arg);
     }

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

@@ -262,6 +262,10 @@ public boolean checkOperandTypes(List<RelDataType> types) {
         // with strings in isComparable
         RelDataType type_l = types.get(i);
         RelDataType type_r = types.get(i + 1);
+        // Rule out IP types from built-in comparable functions
+        if (type_l instanceof ExprIPType || type_r instanceof ExprIPType) {
+          return false;
+        }
         if (!isComparable(type_l, type_r)) {
           return false;
         }