You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2012/03/06 17:07:08 UTC
svn commit: r1297547 - in /lucene/dev/branches/branch_3x: ./ lucene/
lucene/core/src/java/org/apache/lucene/util/packed/
lucene/core/src/test/org/apache/lucene/index/
Author: mikemccand
Date: Tue Mar 6 16:07:08 2012
New Revision: 1297547
URL: http://svn.apache.org/viewvc?rev=1297547&view=rev
Log:
LUCENE-3851: fix test bug; add asserts
Modified:
lucene/dev/branches/branch_3x/ (props changed)
lucene/dev/branches/branch_3x/lucene/ (props changed)
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct16.java
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct32.java
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct64.java
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct8.java
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed32.java
lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed64.java
lucene/dev/branches/branch_3x/lucene/core/src/test/org/apache/lucene/index/TestTermInfosReaderIndex.java
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct16.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct16.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct16.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct16.java Tue Mar 6 16:07:08 2012
@@ -69,6 +69,7 @@ class Direct16 extends PackedInts.Reader
}
public long get(final int index) {
+ assert index >= 0 && index < size();
return 0xFFFFL & values[index];
}
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct32.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct32.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct32.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct32.java Tue Mar 6 16:07:08 2012
@@ -65,6 +65,7 @@ class Direct32 extends PackedInts.Reader
}
public long get(final int index) {
+ assert index >= 0 && index < size();
return 0xFFFFFFFFL & values[index];
}
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct64.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct64.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct64.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct64.java Tue Mar 6 16:07:08 2012
@@ -61,6 +61,7 @@ class Direct64 extends PackedInts.Reader
}
public long get(final int index) {
+ assert index >= 0 && index < size();
return values[index];
}
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct8.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct8.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct8.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Direct8.java Tue Mar 6 16:07:08 2012
@@ -70,6 +70,7 @@ class Direct8 extends PackedInts.ReaderI
}
public long get(final int index) {
+ assert index >= 0 && index < size();
return 0xFFL & values[index];
}
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed32.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed32.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed32.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed32.java Tue Mar 6 16:07:08 2012
@@ -186,6 +186,7 @@ class Packed32 extends PackedInts.Reader
* @return the value at the given index.
*/
public long get(final int index) {
+ assert index >= 0 && index < size();
final long majorBitPos = (long)index * bitsPerValue;
final int elementPos = (int)(majorBitPos >>> BLOCK_BITS); // / BLOCK_SIZE
final int bitPos = (int)(majorBitPos & MOD_MASK); // % BLOCK_SIZE);
Modified: lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed64.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed64.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed64.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/java/org/apache/lucene/util/packed/Packed64.java Tue Mar 6 16:07:08 2012
@@ -177,6 +177,7 @@ class Packed64 extends PackedInts.Reader
* @return the value at the given index.
*/
public long get(final int index) {
+ assert index >= 0 && index < size();
final long majorBitPos = (long)index * bitsPerValue;
final int elementPos = (int)(majorBitPos >>> BLOCK_BITS); // / BLOCK_SIZE
final int bitPos = (int)(majorBitPos & MOD_MASK); // % BLOCK_SIZE);
Modified: lucene/dev/branches/branch_3x/lucene/core/src/test/org/apache/lucene/index/TestTermInfosReaderIndex.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/core/src/test/org/apache/lucene/index/TestTermInfosReaderIndex.java?rev=1297547&r1=1297546&r2=1297547&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/core/src/test/org/apache/lucene/index/TestTermInfosReaderIndex.java (original)
+++ lucene/dev/branches/branch_3x/lucene/core/src/test/org/apache/lucene/index/TestTermInfosReaderIndex.java Tue Mar 6 16:07:08 2012
@@ -63,8 +63,8 @@ public class TestTermInfosReaderIndex ex
NUMBER_OF_FIELDS = atLeast(Math.max(10, 3*termIndexInterval*indexDivisor/NUMBER_OF_DOCUMENTS));
directory = newDirectory();
- // turn off compound file, this test will open some index files directly.
LogMergePolicy mp = newLogMergePolicy();
+ // turn off compound file, this test will open some index files directly.
mp.setUseCompoundFile(false);
config.setMergePolicy(mp);
@@ -156,7 +156,14 @@ public class TestTermInfosReaderIndex ex
// TODO: this test just uses random terms, so this is always possible
assumeTrue("ran out of terms.", termEnum.next());
}
- return termEnum.term();
+ final Term term = termEnum.term();
+ // An indexed term is only written when the term after
+ // it exists, so, if the number of terms is 0 mod
+ // termIndexInterval, the last index term will not be
+ // written; so we require a term after this term
+ // as well:
+ assumeTrue("ran out of terms", termEnum.next());
+ return term;
}
private static void populate(Directory directory, IndexWriterConfig config) throws CorruptIndexException, LockObtainFailedException, IOException {