You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by da...@apache.org on 2017/12/27 15:04:27 UTC

[34/54] [abbrv] lucene-solr:jira/solr-11702: LUCENE-8077: Fixed wrong modulo usage in CheckIndex.

LUCENE-8077: Fixed wrong modulo usage in CheckIndex.


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

Branch: refs/heads/jira/solr-11702
Commit: b5ad3148358cea4f24abaacadbc9a4ae03f1b7c7
Parents: 9f7f76f
Author: Adrien Grand <jp...@gmail.com>
Authored: Wed Dec 20 12:48:04 2017 +0100
Committer: Adrien Grand <jp...@gmail.com>
Committed: Wed Dec 20 12:48:04 2017 +0100

----------------------------------------------------------------------
 lucene/CHANGES.txt                                           | 5 +++++
 lucene/core/src/java/org/apache/lucene/index/CheckIndex.java | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b5ad3148/lucene/CHANGES.txt
----------------------------------------------------------------------
diff --git a/lucene/CHANGES.txt b/lucene/CHANGES.txt
index d8e1237..98cdcd9 100644
--- a/lucene/CHANGES.txt
+++ b/lucene/CHANGES.txt
@@ -94,6 +94,11 @@ Improvements
 
 * LUCENE-8093: TrimFilterFactory implements MultiTermAwareComponent (Alan Woodward)
 
+Bug Fixes
+
+* LUCENE-8077: Fixed bug in how CheckIndex verifies doc-value iterators.
+  (Xiaoshan Sun via Adrien Grand)
+
 ======================= Lucene 7.2.0 =======================
 
 API Changes

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b5ad3148/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java b/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
index 028da24..c676568 100644
--- a/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
+++ b/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
@@ -2195,7 +2195,7 @@ public final class CheckIndex implements Closeable {
           throw new RuntimeException("dv iterator field=" + field + ": doc=" + (doc-1) + " has unstable advanceExact");
         }
 
-        if (i % 1 == 0) {
+        if (i % 2 == 0) {
           int doc2 = it2.nextDoc();
           if (doc != doc2) {
             throw new RuntimeException("dv iterator field=" + field + ": doc=" + doc + " was not found through advance() (got: " + doc2 + ")");