You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2012/04/10 12:44:12 UTC
svn commit: r1311670 -
/lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java
Author: siren
Date: Tue Apr 10 10:44:11 2012
New Revision: 1311670
URL: http://svn.apache.org/viewvc?rev=1311670&view=rev
Log:
SOLR-3237: fix test, add more iterations when waiting for shardId
Modified:
lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java?rev=1311670&r1=1311669&r2=1311670&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/OverseerTest.java Tue Apr 10 10:44:11 2012
@@ -93,7 +93,7 @@ public class OverseerTest extends SolrTe
public void close(){
try {
- deleteNode(ZkStateReader.LIVE_NODES_ZKNODE + "/" + "node1");
+ deleteNode(ZkStateReader.LIVE_NODES_ZKNODE + "/" + nodeName);
zkClient.close();
} catch (InterruptedException e) {
//e.printStackTrace();
@@ -127,7 +127,7 @@ public class OverseerTest extends SolrTe
ZkStateReader.toJSON(coreStates.values().toArray(
new CoreState[coreStates.size()])), true);
- for (int i = 0; i < 10; i++) {
+ for (int i = 0; i < 30; i++) {
String shardId = getShardId(coreName);
if (shardId != null) {
try {
@@ -154,9 +154,9 @@ public class OverseerTest extends SolrTe
collection);
if (slices != null) {
for (Slice slice : slices.values()) {
- if (slice.getShards().containsKey(nodeName + "_" + coreName))
- ;
- return slice.getName();
+ if (slice.getShards().containsKey(nodeName + "_" + coreName)) {
+ return slice.getName();
+ }
}
}
return null;