fixup

Author: pabloem

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/Clamp.java

@@ -33,8 +33,6 @@
  * Clamps the values of all samples to have a lower limit of min and an upper limit of max.
  */
 public class Clamp extends EsqlScalarFunction implements SurrogateExpression {
-    public static final NamedWriteableRegistry.Entry ENTRY = new NamedWriteableRegistry.Entry(Expression.class, "Clamp", Clamp::new);
-
     private final Expression field;
     private final Expression min;
     private final Expression max;
@@ -68,18 +66,9 @@ public Clamp(
         this.max = max;
     }
 
-    private Clamp(StreamInput in) throws IOException {
-        this(
-            Source.readFrom((PlanStreamInput) in),
-            in.readNamedWriteable(Expression.class),
-            in.readNamedWriteable(Expression.class),
-            in.readNamedWriteable(Expression.class)
-        );
-    }
-
     @Override
     public String getWriteableName() {
-        return ENTRY.name;
+        throw new UnsupportedOperationException("Clamp does not support serialization.");
     }
 
     @Override
@@ -135,10 +124,7 @@ protected NodeInfo<? extends Expression> info() {
 
     @Override
     public void writeTo(StreamOutput out) throws IOException {
-        source().writeTo(out);
-        out.writeNamedWriteable(field);
-        out.writeNamedWriteable(min);
-        out.writeNamedWriteable(max);
+        throw new UnsupportedOperationException("Clamp does not support serialization.");
     }
 
     @Override

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/ScalarFunctionWritables.java

@@ -92,7 +92,6 @@ public static List<NamedWriteableRegistry.Entry> getNamedWriteables() {
         entries.add(MonthName.ENTRY);
         entries.add(IpPrefix.ENTRY);
         entries.add(Least.ENTRY);
-        entries.add(Clamp.ENTRY);
         entries.add(ClampMax.ENTRY);
         entries.add(ClampMin.ENTRY);
         entries.add(Left.ENTRY);

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/scalar/conditional/ClampTests.java

@@ -31,7 +31,12 @@ public ClampTests(@Name("TestCase") Supplier<TestCaseSupplier.TestCase> testCase
         this.testCase = testCaseSupplier.get();
     }
 
-    @ParametersFactory
+    @Override
+    protected boolean canSerialize() {
+        return false;
+    }
+
+        @ParametersFactory
     public static Iterable<Object[]> parameters() {
         List<TestCaseSupplier> suppliers = new java.util.ArrayList<>();
         for (DataType stringType : DataType.stringTypes()) {