You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ab...@apache.org on 2017/04/03 18:45:53 UTC

[47/52] [abbrv] lucene-solr:jira/solr-9959: switch to advanceExact

switch to advanceExact


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/99af8302
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/99af8302
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/99af8302

Branch: refs/heads/jira/solr-9959
Commit: 99af830223a4ef69387586531e56338590659f7c
Parents: ef82183
Author: Mike McCandless <mi...@apache.org>
Authored: Sun Apr 2 16:25:09 2017 -0400
Committer: Mike McCandless <mi...@apache.org>
Committed: Sun Apr 2 16:25:09 2017 -0400

----------------------------------------------------------------------
 .../org/apache/lucene/search/DiversifiedTopDocsCollector.java   | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/99af8302/lucene/misc/src/java/org/apache/lucene/search/DiversifiedTopDocsCollector.java
----------------------------------------------------------------------
diff --git a/lucene/misc/src/java/org/apache/lucene/search/DiversifiedTopDocsCollector.java b/lucene/misc/src/java/org/apache/lucene/search/DiversifiedTopDocsCollector.java
index 6b75622..1a087d4 100644
--- a/lucene/misc/src/java/org/apache/lucene/search/DiversifiedTopDocsCollector.java
+++ b/lucene/misc/src/java/org/apache/lucene/search/DiversifiedTopDocsCollector.java
@@ -124,10 +124,7 @@ public abstract class DiversifiedTopDocsCollector extends
     // a leaf reader value when looking up keys
     int leafDocID = addition.doc - docBase;
     long value;
-    if (keys.docID() < leafDocID) {
-      keys.advance(leafDocID);
-    }
-    if (keys.docID() == leafDocID) {
+    if (keys.advanceExact(leafDocID)) {
       value = keys.longValue();
     } else {
       value = 0;