You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2015/06/09 00:16:01 UTC

svn commit: r1684292 - in /lucene/dev/branches/branch_5x: ./ lucene/ lucene/core/ lucene/core/src/java/org/apache/lucene/index/CheckIndex.java lucene/test-framework/ lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java

Author: rmuir
Date: Mon Jun  8 22:16:01 2015
New Revision: 1684292

URL: http://svn.apache.org/r1684292
Log:
LUCENE-6529: add asserts

Modified:
    lucene/dev/branches/branch_5x/   (props changed)
    lucene/dev/branches/branch_5x/lucene/   (props changed)
    lucene/dev/branches/branch_5x/lucene/core/   (props changed)
    lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
    lucene/dev/branches/branch_5x/lucene/test-framework/   (props changed)
    lucene/dev/branches/branch_5x/lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java

Modified: lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java?rev=1684292&r1=1684291&r2=1684292&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java (original)
+++ lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/CheckIndex.java Mon Jun  8 22:16:01 2015
@@ -1602,6 +1602,10 @@ public class CheckIndex implements Close
             for(int i=seekCount-1;i>=0;i--) {
               long ord = i*(termCount/seekCount);
               termsEnum.seekExact(ord);
+              long actualOrd = termsEnum.ord();
+              if (actualOrd != ord) {
+                throw new RuntimeException("seek to ord " + ord + " returned ord " + actualOrd);
+              }
               seekTerms[i] = BytesRef.deepCopyOf(termsEnum.term());
             }
             

Modified: lucene/dev/branches/branch_5x/lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java?rev=1684292&r1=1684291&r2=1684292&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java (original)
+++ lucene/dev/branches/branch_5x/lucene/test-framework/src/java/org/apache/lucene/index/RandomPostingsTester.java Mon Jun  8 22:16:01 2015
@@ -1133,6 +1133,9 @@ public class RandomPostingsTester {
       } else {
         termsEnum.seekExact(fieldAndTerm.term, termState);
       }
+      
+      // check we really seeked to the right place
+      assertEquals(fieldAndTerm.term, termsEnum.term());
 
       long termOrd;
       if (supportsOrds) {