You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2010/07/28 07:27:04 UTC
svn commit: r979951 -
/lucene/dev/trunk/solr/src/test/org/apache/solr/handler/TestReplicationHandler.java
Author: markrmiller
Date: Wed Jul 28 05:27:03 2010
New Revision: 979951
URL: http://svn.apache.org/viewvc?rev=979951&view=rev
Log:
retry to prevent false failures
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=979951&r1=979950&r2=979951&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 Wed Jul 28 05:27:03 2010
@@ -449,8 +449,18 @@ public class TestReplicationHandler exte
slaveClient.commit(true, true);
//this doc is added to slave so it should show an item w/ that result
- NamedList slaveQueryRsp = rQuery(1, "id:555", slaveClient);
- SolrDocumentList slaveQueryResult = (SolrDocumentList) slaveQueryRsp.get("response");
+ SolrDocumentList slaveQueryResult = null;
+ NamedList slaveQueryRsp;
+ // try a few times in case tests are running slow
+ for (int i = 0; i < 3; i++) {
+ slaveQueryRsp = rQuery(1, "id:555", slaveClient);
+ slaveQueryResult = (SolrDocumentList) slaveQueryRsp.get("response");
+ if (slaveQueryResult.getNumFound() > 0) {
+ break;
+ }
+ Thread.sleep(100);
+ }
+ assertNotNull(slaveQueryResult);
assertEquals(1, slaveQueryResult.getNumFound());
masterUrl = "http://localhost:" + masterJetty.getLocalPort() + "/solr/replication?command=enableReplication";