Making version of lucene k-nn engine match lucene current version (#691…

…) (#712) * Making version of lucene k-nn engine match lucene current version Signed-off-by: Martin Gaievski <[email protected]> (cherry picked from commit 577205e) Co-authored-by: Martin Gaievski <[email protected]>
opensearch-project · Jan 6, 2023 · dd7d0c5 · dd7d0c5
1 parent 084f94f
commit dd7d0c5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/src/main/java/org/opensearch/knn/index/util/Lucene.java b/src/main/java/org/opensearch/knn/index/util/Lucene.java
@@ -44,7 +44,7 @@ public class Lucene extends JVMLibrary {
         ).addSpaces(SpaceType.L2, SpaceType.COSINESIMIL).build()
     );
 
-    final static Lucene INSTANCE = new Lucene(METHODS, Version.LUCENE_9_2_0.toString());
+    final static Lucene INSTANCE = new Lucene(METHODS, Version.LATEST.toString());
 
     /**
      * Constructor

diff --git a/src/test/java/org/opensearch/knn/index/util/KNNEngineTests.java b/src/test/java/org/opensearch/knn/index/util/KNNEngineTests.java
@@ -14,6 +14,8 @@ public class KNNEngineTests extends KNNTestCase {
      */
     public void testDelegateLibraryFunctions() {
         assertEquals(Nmslib.INSTANCE.getVersion(), KNNEngine.NMSLIB.getVersion());
+        assertEquals(Faiss.INSTANCE.getVersion(), KNNEngine.FAISS.getVersion());
+        assertEquals(Lucene.INSTANCE.getVersion(), KNNEngine.LUCENE.getVersion());
     }
 
     /**

diff --git a/src/test/java/org/opensearch/knn/index/util/LuceneTests.java b/src/test/java/org/opensearch/knn/index/util/LuceneTests.java
@@ -5,6 +5,7 @@
 
 package org.opensearch.knn.index.util;
 
+import org.apache.lucene.util.Version;
 import org.opensearch.common.xcontent.XContentBuilder;
 import org.opensearch.common.xcontent.XContentFactory;
 import org.opensearch.knn.KNNTestCase;
@@ -108,4 +109,9 @@ public void testSetInitialized() {
         luceneLibrary.setInitialized(true);
         assertTrue(luceneLibrary.isInitialized());
     }
+
+    public void testVersion() {
+        Lucene luceneInstance = Lucene.INSTANCE;
+        assertEquals(Version.LATEST.toString(), luceneInstance.getVersion());
+    }
 }