Updating NPE bug fixes for opensearch-project#2556

Signed-off-by: Arun Ganesh <[email protected]>
oaganesh · Mar 5, 2025 · fc14165 · fc14165
1 parent e7d5304
commit fc14165
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/jni/external/faiss b/jni/external/faiss
diff --git a/jni/external/nmslib b/jni/external/nmslib
diff --git a/src/main/java/org/opensearch/knn/index/mapper/KNNVectorFieldMapper.java b/src/main/java/org/opensearch/knn/index/mapper/KNNVectorFieldMapper.java
@@ -161,7 +161,13 @@ public static class Builder extends ParametrizedFieldMapper.Builder {
             b.startObject(n);
             v.toXContent(b, ToXContent.EMPTY_PARAMS);
             b.endObject();
-        }), m -> m.getMethodComponentContext().getName());
+        }), m -> {
+            if (m == null) {
+                throw new IllegalArgumentException("Mapping update for knn_vector fields is not supported. " +
+                        "Cannot update mapping without the original method configuration.");
+            }
+            return m.getMethodComponentContext().getName();
+        });
 
         protected final Parameter<String> mode = Parameter.restrictedStringParam(
             KNNConstants.MODE_PARAMETER,