svn commit: r1327192 - /lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java

[yo...@apache.org]

Author: yonik
Date: Tue Apr 17 17:26:49 2012
New Revision: 1327192

URL: http://svn.apache.org/viewvc?rev=1327192&view=rev
Log:
SOLR-3362: log unexpected term for facet refinement

Modified:
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java?rev=1327192&r1=1327191&r2=1327192&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/FacetComponent.java Tue Apr 17 17:26:49 2012
@@ -30,6 +30,8 @@ import org.apache.solr.common.util.StrUt
 import org.apache.solr.request.SimpleFacets;
 import org.apache.solr.schema.FieldType;
 import org.apache.solr.search.QueryParsing;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
 import java.net.URL;
@@ -43,6 +45,8 @@ import java.util.*;
  */
 public class FacetComponent extends SearchComponent
 {
+  public static Logger log = LoggerFactory.getLogger(FacetComponent.class);
+
   public static final String COMPONENT_NAME = "facet";
 
   static final String PIVOT_KEY = "facet_pivot";
@@ -486,6 +490,15 @@ public class FacetComponent extends Sear
           String name = shardCounts.getName(j);
           long count = ((Number)shardCounts.getVal(j)).longValue();
           ShardFacetCount sfc = dff.counts.get(name);
+          if (sfc == null) {
+            // we got back a term we didn't ask for?
+            log.error("Unexpected term returned for facet refining. key=" + key + " term='" + name + "'"
+              + "\n\trequest params=" + sreq.params
+              + "\n\ttoRefine=" + dff._toRefine
+              + "\n\tresponse=" + shardCounts
+            );
+            continue;
+          }
           sfc.count += count;
         }
       }