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 2015/04/16 20:17:42 UTC

svn commit: r1674125 - in /lucene/dev/branches/branch_5x: ./ lucene/ lucene/backward-codecs/ lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java

Author: mikemccand
Date: Thu Apr 16 18:17:42 2015
New Revision: 1674125

URL: http://svn.apache.org/r1674125
Log:
fix test bug: this test relies on docid ordering

Modified:
    lucene/dev/branches/branch_5x/   (props changed)
    lucene/dev/branches/branch_5x/lucene/   (props changed)
    lucene/dev/branches/branch_5x/lucene/backward-codecs/   (props changed)
    lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java

Modified: lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java?rev=1674125&r1=1674124&r2=1674125&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java (original)
+++ lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/index/TestMaxPositionInOldIndex.java Thu Apr 16 18:17:42 2015
@@ -90,13 +90,17 @@ public class BuildMaxPositionIndex {
     }
 
     // Also confirm merging detects this:
-    IndexWriter w = new IndexWriter(dir, newIndexWriterConfig().setMergeScheduler(new SerialMergeScheduler()));
+    IndexWriterConfig iwc = newIndexWriterConfig();
+    iwc.setMergeScheduler(new SerialMergeScheduler());
+    iwc.setMergePolicy(newLogMergePolicy());
+    IndexWriter w = new IndexWriter(dir, iwc);
     w.addDocument(new Document());
     try {
       w.forceMerge(1);
     } catch (CorruptIndexException cie) {
       // SerialMergeScheduler
-      assertTrue(cie.getMessage().contains("position=2147483647 is too large (> IndexWriter.MAX_POSITION=2147483519), field=\"foo\" doc=0 (resource=PerFieldPostings(segment=_0 formats=1)"));
+      assertTrue("got message " + cie.getMessage(),
+                 cie.getMessage().contains("position=2147483647 is too large (> IndexWriter.MAX_POSITION=2147483519), field=\"foo\" doc=0 (resource=PerFieldPostings(segment=_0 formats=1)"));
     }
 
     w.close();