Add back public interface, now deprecated

1 files changed, 15 insertions, 0 deletions

diff --git a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ExpressionNode.java b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ExpressionNode.java
index c74b90d2ea5..84a428d48be 100755
--- a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ExpressionNode.java
+++ b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ExpressionNode.java
@@ -36,6 +36,7 @@ public abstract class ExpressionNode implements Serializable {
     /**
      * Returns a script instance of this based on the supplied script functions.
      *
+     * @param builder the StringBuilder that will be appended to
      * @param context the serialization context
      * @param path the call path to this, used for cycle detection, or null if this is a root
      * @param parent the parent node of this, or null if it a root
@@ -44,6 +45,20 @@ public abstract class ExpressionNode implements Serializable {
     public abstract StringBuilder toString(StringBuilder builder, SerializationContext context, Deque<String> path, CompositeNode parent);
 
     /**
+     * Returns a script instance of this based on the supplied script functions.
+     * @Deprecated, use the faster one that takes and returns a StringBuilder instead.
+     *
+     * @param context the serialization context
+     * @param path the call path to this, used for cycle detection, or null if this is a root
+     * @param parent the parent node of this, or null if it a root
+     * @return the main script, referring to script instances.
+     */
+    @Deprecated
+    public String toString(SerializationContext context, Deque<String> path, CompositeNode parent) {
+        return toString(new StringBuilder(), context, path, parent).toString();
+    }
+
+    /**
      * Returns the type this will return if evaluated with the given context.
      *
      * @param context the variable type bindings to use for this evaluation