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());