LuXugang commented on code in PR #12017:
URL: https://github.com/apache/lucene/pull/12017#discussion_r1049163040
##########
lucene/core/src/test/org/apache/lucene/search/TestBooleanQuery.java:
##########
@@ -1015,6 +1015,80 @@ public void testDisjunctionRandomClausesMatchesCount()
throws Exception {
}
}
+ public void testAggressiveMatchCount() throws IOException {
+ Directory dir = newDirectory();
+ IndexWriter writer = new IndexWriter(dir, new IndexWriterConfig());
+ Document doc = new Document();
+ LongPoint longPoint = new LongPoint("long", 3L, 4L, 5L);
+ doc.add(longPoint);
+ StringField stringField = new StringField("string", "abc", Store.NO);
+ doc.add(stringField);
+ writer.addDocument(doc);
+ longPoint.setLongValues(10L, 11L, 12L);
+ stringField.setStringValue("xyz");
+ writer.addDocument(doc);
+
+ IndexReader reader = DirectoryReader.open(writer);
+ writer.close();
+ IndexSearcher searcher = new IndexSearcher(reader);
+
+ long[] lower = new long[] {4L, 5L, 6L};
+ long[] upper = new long[] {9L, 10L, 11L};
+ Query unknownCountQuery = LongPoint.newRangeQuery("long", lower, upper);
+ assert reader.leaves().size() == 1;
+ assert searcher
+ .createWeight(unknownCountQuery, ScoreMode.COMPLETE, 1f)
+ .count(reader.leaves().get(0))
+ == -1;
+
+ Query query =
+ new BooleanQuery.Builder()
+ .add(new TermQuery(new Term("string", "xyz")), Occur.MUST)
+ .add(unknownCountQuery, Occur.MUST_NOT)
+ .add(new MatchAllDocsQuery(), Occur.MUST_NOT)
+ .build();
+ Weight weight = searcher.createWeight(query, ScoreMode.COMPLETE, 1f);
+ // count of the first MUST_NOT clause is unknown, but the second MUST_NOT
clause matches all
+ // docs
+ assertEquals(0, weight.count(reader.leaves().get(0)));
+
+ query =
+ new BooleanQuery.Builder()
+ .add(new TermQuery(new Term("string", "xyz")), Occur.MUST)
+ .add(unknownCountQuery, Occur.MUST_NOT)
+ .add(new TermQuery(new Term("string", "abc")), Occur.MUST_NOT)
+ .build();
+ weight = searcher.createWeight(query, ScoreMode.COMPLETE, 1f);
+ // count of the first MUST_NOT clause is unknown, though the second
MUST_NOT clause matche one
+ // doc, we can't figure out the number of
+ // docs
+ assertEquals(-1, weight.count(reader.leaves().get(0)));
+
+ // test pure disConjunction
Review Comment:
addressed in
https://github.com/apache/lucene/pull/12017/commits/272dec54a59dedc666cf8311c09830c94b3c5369
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
