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 2013/01/02 01:47:23 UTC

svn commit: r1427606 - /lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java

Author: markrmiller
Date: Wed Jan  2 00:47:23 2013
New Revision: 1427606

URL: http://svn.apache.org/viewvc?rev=1427606&view=rev
Log:
tests: only do this check if there are actually different docs

Modified:
    lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java

Modified: lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java?rev=1427606&r1=1427605&r2=1427606&view=diff
==============================================================================
--- lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java (original)
+++ lucene/dev/trunk/solr/test-framework/src/java/org/apache/solr/cloud/AbstractFullDistribZkTestBase.java Wed Jan  2 00:47:23 2013
@@ -1023,15 +1023,19 @@ public abstract class AbstractFullDistri
         ids.append(" "+doc.get("id"));
       }
       ids.append(")");
-
-      // get versions for those ids that don't match
-      q = params("q",ids.toString(),"rows","100000", "fl","id,_version_", "sort","id asc",
-                 "tests","checkShardConsistency(vsControl)/getVers");    // add a tag to aid in debugging via logs
-
-      SolrDocumentList a = controlClient.query(q).getResults();
-      SolrDocumentList b = cloudClient.query(q).getResults();
-
-      log.error("controlClient :" + a + "\n\tcloudClient :" + b);
+      
+      if (ids.length() > 0) {
+        // get versions for those ids that don't match
+        q = params("q", ids.toString(), "rows", "100000", "fl", "id,_version_",
+            "sort", "id asc", "tests",
+            "checkShardConsistency(vsControl)/getVers"); // add a tag to aid in
+                                                         // debugging via logs
+        
+        SolrDocumentList a = controlClient.query(q).getResults();
+        SolrDocumentList b = cloudClient.query(q).getResults();
+        
+        log.error("controlClient :" + a + "\n\tcloudClient :" + b);
+      }
 
       fail(msg);
     }