Fix bwc issue in remote prediction (#2085) (#2097)

* Fix bwc issue in remote prediction Signed-off-by: Sicheng Song <[email protected]> * Fix typo Signed-off-by: Sicheng Song <[email protected]> * Re-order the function name type to the right order Signed-off-by: Sicheng Song <[email protected]> --------- Signed-off-by: Sicheng Song <[email protected]> (cherry picked from commit b3dfabc) Co-authored-by: Sicheng Song <[email protected]>
opensearch-project · Feb 13, 2024 · 65ef97e · 65ef97e
1 parent 8691863
commit 65ef97e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/common/src/main/java/org/opensearch/ml/common/FunctionName.java b/common/src/main/java/org/opensearch/ml/common/FunctionName.java
@@ -8,6 +8,9 @@
 import java.util.HashSet;
 import java.util.Set;
 
+// Please strictly add new FunctionName to the last line
+// and avoid altering the order of the existing FunctionName,
+// or it will break the backward compatibility!
 public enum FunctionName {
     LINEAR_REGRESSION,
     KMEANS,
@@ -20,11 +23,11 @@ public enum FunctionName {
     RCF_SUMMARIZE,
     LOGISTIC_REGRESSION,
     TEXT_EMBEDDING,
-    TEXT_SIMILARITY,
-    SPARSE_ENCODING,
-    SPARSE_TOKENIZE,
     METRICS_CORRELATION,
     REMOTE,
+    SPARSE_ENCODING,
+    SPARSE_TOKENIZE,
+    TEXT_SIMILARITY,
     AGENT;
 
     public static FunctionName from(String value) {

diff --git a/common/src/main/java/org/opensearch/ml/common/dataset/MLInputDataType.java b/common/src/main/java/org/opensearch/ml/common/dataset/MLInputDataType.java
@@ -5,10 +5,13 @@
 
 package org.opensearch.ml.common.dataset;
 
+// Please strictly add new MLInputDataType to the last line
+// and avoid altering the order of the existing MLInputDataType,
+// or it will break the backward compatibility!
 public enum MLInputDataType {
     SEARCH_QUERY,
     DATA_FRAME,
     TEXT_DOCS,
-    TEXT_SIMILARITY,
-    REMOTE
+    REMOTE,
+    TEXT_SIMILARITY
 }