Oh -- sorry, I guess I forgot to backport. Thanks for tracking it down - I'll push to branch_9x shortly
On Wed, Aug 31, 2022 at 10:25 AM Robert Muir <rcm...@gmail.com> wrote: > > can we backport to 9.x if you get a chance? I'm still seeing this test > trip in 9.x jenkins builds. > > > On Mon, Aug 29, 2022 at 11:50 AM <soko...@apache.org> wrote: > > > > This is an automated email from the ASF dual-hosted git repository. > > > > sokolov pushed a commit to branch main > > in repository https://gitbox.apache.org/repos/asf/lucene.git > > > > > > The following commit(s) were added to refs/heads/main by this push: > > new 61ef031f7fa SimpleText knn vectors; fix searchExhaustively and > > suppress a byte format test case (#11725) > > 61ef031f7fa is described below > > > > commit 61ef031f7fa3abdd7c8c2f36db71ad2289b66131 > > Author: Michael Sokolov <soko...@falutin.net> > > AuthorDate: Mon Aug 29 11:49:52 2022 -0400 > > > > SimpleText knn vectors; fix searchExhaustively and suppress a byte > > format test case (#11725) > > --- > > .../apache/lucene/codecs/simpletext/SimpleTextKnnVectorsReader.java | 6 > > +++--- > > lucene/core/src/test/org/apache/lucene/document/TestField.java | 4 > > ++++ > > 2 files changed, 7 insertions(+), 3 deletions(-) > > > > diff --git > > a/lucene/codecs/src/java/org/apache/lucene/codecs/simpletext/SimpleTextKnnVectorsReader.java > > > > b/lucene/codecs/src/java/org/apache/lucene/codecs/simpletext/SimpleTextKnnVectorsReader.java > > index e4b0ceb5916..10700f5de6f 100644 > > --- > > a/lucene/codecs/src/java/org/apache/lucene/codecs/simpletext/SimpleTextKnnVectorsReader.java > > +++ > > b/lucene/codecs/src/java/org/apache/lucene/codecs/simpletext/SimpleTextKnnVectorsReader.java > > @@ -41,7 +41,6 @@ import org.apache.lucene.store.BufferedChecksumIndexInput; > > import org.apache.lucene.store.ChecksumIndexInput; > > import org.apache.lucene.store.IOContext; > > import org.apache.lucene.store.IndexInput; > > -import org.apache.lucene.util.BitSet; > > import org.apache.lucene.util.Bits; > > import org.apache.lucene.util.BytesRef; > > import org.apache.lucene.util.BytesRefBuilder; > > @@ -187,8 +186,9 @@ public class SimpleTextKnnVectorsReader extends > > KnnVectorsReader { > > @Override > > public TopDocs searchExhaustively( > > String field, float[] target, int k, DocIdSetIterator acceptDocs) > > throws IOException { > > - int numDocs = (int) acceptDocs.cost(); > > - return search(field, target, k, BitSet.of(acceptDocs, numDocs), > > Integer.MAX_VALUE); > > + FieldInfo info = readState.fieldInfos.fieldInfo(field); > > + VectorSimilarityFunction vectorSimilarity = > > info.getVectorSimilarityFunction(); > > + return exhaustiveSearch(getVectorValues(field), acceptDocs, > > vectorSimilarity, target, k); > > } > > > > @Override > > diff --git a/lucene/core/src/test/org/apache/lucene/document/TestField.java > > b/lucene/core/src/test/org/apache/lucene/document/TestField.java > > index 781f2b613c6..6aa5518f33b 100644 > > --- a/lucene/core/src/test/org/apache/lucene/document/TestField.java > > +++ b/lucene/core/src/test/org/apache/lucene/document/TestField.java > > @@ -20,6 +20,7 @@ import static > > org.apache.lucene.search.DocIdSetIterator.NO_MORE_DOCS; > > > > import java.io.StringReader; > > import java.nio.charset.StandardCharsets; > > +import org.apache.lucene.codecs.Codec; > > import org.apache.lucene.index.DirectoryReader; > > import org.apache.lucene.index.IndexReader; > > import org.apache.lucene.index.IndexWriter; > > @@ -513,6 +514,9 @@ public class TestField extends LuceneTestCase { > > } > > > > public void testKnnVectorField() throws Exception { > > + if (Codec.getDefault().getName().equals("SimpleText")) { > > + return; > > + } > > try (Directory dir = newDirectory(); > > IndexWriter w = new IndexWriter(dir, newIndexWriterConfig())) { > > Document doc = new Document(); > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org > For additional commands, e-mail: dev-h...@lucene.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
