name

Author: JingsongLi

paimon-common/src/main/java/org/apache/paimon/predicate/CastTransform.java

@@ -61,6 +61,11 @@ public static Optional<Transform> tryCreate(FieldRef fieldRef, DataType type) {
         }
     }
 
+    @Override
+    public String name() {
+        return "CAST";
+    }
+
     @Override
     public List<Object> inputs() {
         return Collections.singletonList(fieldRef);

paimon-common/src/main/java/org/apache/paimon/predicate/ConcatTransform.java

@@ -31,6 +31,11 @@ public ConcatTransform(List<Object> inputs) {
         super(inputs);
     }
 
+    @Override
+    public String name() {
+        return "CONCAT";
+    }
+
     @Override
     public BinaryString transform(List<BinaryString> inputs) {
         return BinaryString.concat(inputs);

paimon-common/src/main/java/org/apache/paimon/predicate/ConcatWsTransform.java

@@ -34,6 +34,11 @@ public ConcatWsTransform(List<Object> inputs) {
         checkArgument(inputs.size() >= 2);
     }
 
+    @Override
+    public String name() {
+        return "CONCAT_WS";
+    }
+
     @Override
     public BinaryString transform(List<BinaryString> inputs) {
         BinaryString separator = inputs.get(0);

paimon-common/src/main/java/org/apache/paimon/predicate/FieldRef.java

@@ -71,6 +71,6 @@ public int hashCode() {
 
     @Override
     public String toString() {
-        return "FieldRef{" + "index=" + index + ", name='" + name + '\'' + ", type=" + type + '}';
+        return name;
     }
 }

paimon-common/src/main/java/org/apache/paimon/predicate/FieldTransform.java

@@ -39,6 +39,11 @@ public FieldTransform(FieldRef fieldRef) {
         this.fieldRef = fieldRef;
     }
 
+    @Override
+    public String name() {
+        return "FIELD_REF";
+    }
+
     public FieldRef fieldRef() {
         return fieldRef;
     }
@@ -80,6 +85,6 @@ public int hashCode() {
 
     @Override
     public String toString() {
-        return "FieldTransform{" + "fieldRef=" + fieldRef + '}';
+        return fieldRef.name();
     }
 }

paimon-common/src/main/java/org/apache/paimon/predicate/LeafPredicate.java

@@ -172,28 +172,18 @@ public int hashCode() {
 
     @Override
     public String toString() {
-        if (fieldRefOptional().isPresent()) {
-            String literalsStr;
-            int literalsSize = literals == null ? 0 : literals.size();
-            if (literalsSize == 0) {
-                literalsStr = "";
-            } else if (literalsSize == 1) {
-                literalsStr = Objects.toString(literals.get(0));
-            } else {
-                literalsStr = StringUtils.truncatedString(literals, "[", ", ", "]");
-            }
-            return literalsStr.isEmpty()
-                    ? function + "(" + fieldName() + ")"
-                    : function + "(" + fieldName() + ", " + literalsStr + ")";
+        String literalsStr;
+        int literalsSize = literals == null ? 0 : literals.size();
+        if (literalsSize == 0) {
+            literalsStr = "";
+        } else if (literalsSize == 1) {
+            literalsStr = Objects.toString(literals.get(0));
+        } else {
+            literalsStr = StringUtils.truncatedString(literals, "[", ", ", "]");
         }
-        return "{"
-                + "transform="
-                + transform
-                + ", function="
-                + function
-                + ", literals="
-                + literals
-                + '}';
+        return literalsStr.isEmpty()
+                ? function + "(" + transform + ")"
+                : function + "(" + transform + ", " + literalsStr + ")";
     }
 
     private ListSerializer<Object> literalsSerializer() {
@@ -236,6 +226,7 @@ public String fieldName() {
     /** Use {@link #fieldRefOptional()} instead. */
     @Deprecated
     public FieldRef fieldRef() {
+        //noinspection OptionalGetWithoutIsPresent
         return fieldRefOptional().get();
     }
 

paimon-common/src/main/java/org/apache/paimon/predicate/StringTransform.java

@@ -26,6 +26,7 @@
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
+import java.util.stream.Collectors;
 
 import static org.apache.paimon.types.DataTypeFamily.CHARACTER_STRING;
 import static org.apache.paimon.utils.Preconditions.checkArgument;
@@ -95,6 +96,8 @@ public int hashCode() {
 
     @Override
     public String toString() {
-        return getClass().getSimpleName() + "{" + "inputs=" + inputs + '}';
+        List<String> inputs =
+                this.inputs.stream().map(Object::toString).collect(Collectors.toList());
+        return name() + "(" + String.join(", ", inputs) + ')';
     }
 }

paimon-common/src/main/java/org/apache/paimon/predicate/Transform.java

@@ -27,6 +27,8 @@
 /** Represents a transform function. */
 public interface Transform extends Serializable {
 
+    String name();
+
     List<Object> inputs();
 
     DataType outputType();

paimon-common/src/main/java/org/apache/paimon/predicate/UpperTransform.java

@@ -34,6 +34,11 @@ public UpperTransform(List<Object> inputs) {
         checkArgument(inputs.size() == 1);
     }
 
+    @Override
+    public String name() {
+        return "UPPER";
+    }
+
     @Override
     public BinaryString transform(List<BinaryString> inputs) {
         BinaryString string = inputs.get(0);