svn commit: r1643014 - in /lucene/dev/branches/branch_5x: ./ solr/ solr/core/ solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java

[an...@apache.org]

Author: anshum
Date: Tue Dec  2 21:51:17 2014
New Revision: 1643014

URL: http://svn.apache.org/r1643014
Log:
SOLR-6755: Adding a try/catch block and extra logging to get more details when this test fails next (merge from trunk)

Modified:
    lucene/dev/branches/branch_5x/   (props changed)
    lucene/dev/branches/branch_5x/solr/   (props changed)
    lucene/dev/branches/branch_5x/solr/core/   (props changed)
    lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java

Modified: lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java?rev=1643014&r1=1643013&r2=1643014&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java (original)
+++ lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/mlt/CloudMLTQParserTest.java Tue Dec  2 21:51:17 2014
@@ -23,12 +23,17 @@ import org.apache.solr.common.SolrDocume
 import org.apache.solr.common.SolrDocumentList;
 import org.apache.solr.common.params.CommonParams;
 import org.apache.solr.common.params.ModifiableSolrParams;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashSet;
 
 public class CloudMLTQParserTest extends AbstractFullDistribZkTestBase {
+
+  static Logger log = LoggerFactory.getLogger(CloudMLTQParserTest.class);
+  
   public CloudMLTQParserTest() {
     fixShardCount = true;
     shardCount = 2;
@@ -125,13 +130,20 @@ public class CloudMLTQParserTest extends
     String expectedQueryString = "lowerfilt:over lowerfilt:fox lowerfilt:lazy lowerfilt:brown "
         + "lowerfilt:jumped lowerfilt:red lowerfilt:dogs. lowerfilt:quote lowerfilt:the";
     
-    ArrayList<String> actualParsedQueries = (ArrayList<String>) queryResponse
-        .getDebugMap().get("parsedquery");
-    
-    for(int counter=0; counter < actualParsedQueries.size(); counter++) {
-      assertTrue("Parsed queries aren't equal",
-          compareParsedQueryStrings(expectedQueryString,
-          actualParsedQueries.get(counter)));
+    try {
+      ArrayList<String> actualParsedQueries = (ArrayList<String>) queryResponse
+          .getDebugMap().get("parsedquery");
+
+      for (int counter = 0; counter < actualParsedQueries.size(); counter++) {
+        assertTrue("Parsed queries aren't equal",
+            compareParsedQueryStrings(expectedQueryString,
+                actualParsedQueries.get(counter)));
+      }
+    } catch (ClassCastException ex) {
+      // TODO: Adding this to just track a rare test failure.
+      // Once SOLR-6755 is resolved, this should be removed.
+      log.info("QueryResponse.debugMap: {}", queryResponse.getDebugMap().toString());
+      log.info("ClusterState: {}", cloudClient.getZkStateReader().getClusterState().toString());
     }
 
     // Assert that {!mlt}id does not throw an exception i.e. implicitly, only fields that are stored + have explicit