You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by yo...@apache.org on 2015/09/04 18:44:17 UTC
svn commit: r1701291 -
/lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java
Author: yonik
Date: Fri Sep 4 16:44:17 2015
New Revision: 1701291
URL: http://svn.apache.org/r1701291
Log:
SOLR-8007: tests: fix TestSearcherReuse by avoiding background merges
Modified:
lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java?rev=1701291&r1=1701290&r2=1701291&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestSearcherReuse.java Fri Sep 4 16:44:17 2015
@@ -89,7 +89,10 @@ public class TestSearcherReuse extends S
assertU(commit());
}
}
- assertU(commit());
+
+ // with random merge policies, a regular commit can cause a segment to be flushed that can kick off a background merge
+ // that can cause a later commit to actually see changes and open a new searcher. This should not be possible with optimize
+ assertU(optimize());
// seed a single query into the cache
assertQ(req("*:*"), "//*[@numFound='"+numDocs+"']");