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 2012/07/30 21:09:33 UTC

svn commit: r1367229 - in /lucene/dev/branches/branch_4x: ./ solr/ solr/core/ solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java

Author: markrmiller
Date: Mon Jul 30 19:09:33 2012
New Revision: 1367229

URL: http://svn.apache.org/viewvc?rev=1367229&view=rev
Log:
enhance test and turn on compare to control

Modified:
    lucene/dev/branches/branch_4x/   (props changed)
    lucene/dev/branches/branch_4x/solr/   (props changed)
    lucene/dev/branches/branch_4x/solr/core/   (props changed)
    lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java

Modified: lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java?rev=1367229&r1=1367228&r2=1367229&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/cloud/SyncSliceTest.java Mon Jul 30 19:09:33 2012
@@ -76,7 +76,7 @@ public class SyncSliceTest extends FullS
   public SyncSliceTest() {
     super();
     sliceCount = 1;
-    shardCount = TEST_NIGHTLY ? 7 : 3;
+    shardCount = TEST_NIGHTLY ? 7 : 4;
   }
   
   @Override
@@ -184,14 +184,14 @@ public class SyncSliceTest extends FullS
     waitForRecoveriesToFinish(false);
     
     skipServers = getRandomOtherJetty(leaderJetty, null);
-    
+    skipServers.addAll( getRandomOtherJetty(leaderJetty, null));
     // skip list should be 
     
     //System.out.println("leader:" + leaderJetty.url);
     //System.out.println("skip list:" + skipServers);
     
-    // we are skipping the leader and one node
-    assertEquals(1, skipServers.size());
+    // we are skipping  one nodes
+    assertEquals(2, skipServers.size());
     
     // more docs than can peer sync
     for (int i = 0; i < 300; i++) {
@@ -222,15 +222,8 @@ public class SyncSliceTest extends FullS
     Thread.sleep(4000);
     
     waitForRecoveriesToFinish(false);
-    
-    
-    // TODO: for now, we just check consistency -
-    // there will be 305 or 5 docs depending on who
-    // becomes the leader - eventually we want that to
-    // always be the 305
-    //checkShardConsistency(true, true);
-    checkShardConsistency(false, true);
-    
+
+    checkShardConsistency(true, true);
   }
 
   private List<String> getRandomJetty() {