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 2010/07/31 17:42:08 UTC

svn commit: r981068 - /lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java

Author: yonik
Date: Sat Jul 31 15:42:07 2010
New Revision: 981068

URL: http://svn.apache.org/viewvc?rev=981068&view=rev
Log:
TestReplicationHandler - fix testReplicateAfterWrite2Slave by retrying commit also

Modified:
    lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java

Modified: lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java?rev=981068&r1=981067&r2=981068&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java (original)
+++ lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java Sat Jul 31 15:42:07 2010
@@ -448,6 +448,7 @@ public class TestReplicationHandler exte
     index(slaveClient, "id", 555, "name", "name = " + 555);
     slaveClient.commit(true, true);
 
+
     //this doc is added to slave so it should show an item w/ that result
     SolrDocumentList slaveQueryResult = null;
     NamedList slaveQueryRsp;
@@ -459,6 +460,10 @@ public class TestReplicationHandler exte
         break;
       }
       Thread.sleep(100);
+
+      // The last commit may have failed due to exceeding max warming searchers.
+      // We'll try again.
+      slaveClient.commit(true, true);
     }
     assertNotNull(slaveQueryResult);
     assertEquals(1, slaveQueryResult.getNumFound());