You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ho...@apache.org on 2012/08/15 21:44:27 UTC
svn commit: r1373598 - in /lucene/dev/trunk/solr: CHANGES.txt
core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java
core/src/test/org/apache/solr/handler/component/StatsComponentTest.java
Author: hossman
Date: Wed Aug 15 19:44:27 2012
New Revision: 1373598
URL: http://svn.apache.org/viewvc?rev=1373598&view=rev
Log:
SOLR-2981: Fixed StatsComponent to no longer return duplicated information when requesting multiple stats.facet fields
Modified:
lucene/dev/trunk/solr/CHANGES.txt
lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java
lucene/dev/trunk/solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java
Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1373598&r1=1373597&r2=1373598&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Wed Aug 15 19:44:27 2012
@@ -66,6 +66,10 @@ Bug Fixes
* SOLR-3730: Rollback is not implemented quite right and can cause corner case fails in
SolrCloud tests. (rmuir, Mark Miller)
+* SOLR-2981: Fixed StatsComponent to no longer return duplicated information
+ when requesting multiple stats.facet fields.
+ (Roman Kliewer via hossman)
+
Other Changes
----------------------
Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java?rev=1373598&r1=1373597&r2=1373598&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/StatsValuesFactory.java Wed Aug 15 19:44:27 2012
@@ -182,8 +182,8 @@ abstract class AbstractStatsValues<T> im
for (Map.Entry<String, StatsValues> e2 : entry.getValue().entrySet()) {
nl2.add(e2.getKey(), e2.getValue().getStatsValues());
}
- res.add(FACETS, nl);
}
+ res.add(FACETS, nl);
return res;
}
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java?rev=1373598&r1=1373597&r2=1373598&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java Wed Aug 15 19:44:27 2012
@@ -228,32 +228,35 @@ public class StatsComponentTest extends
}
public void doTestFacetStatisticsResult(String f) throws Exception {
- assertU(adoc("id", "1", f, "10", "active_s", "true"));
- assertU(adoc("id", "2", f, "20", "active_s", "true"));
- assertU(adoc("id", "3", f, "30", "active_s", "false"));
- assertU(adoc("id", "4", f, "40", "active_s", "false"));
+ assertU(adoc("id", "1", f, "10", "active_s", "true", "other_s", "foo"));
+ assertU(adoc("id", "2", f, "20", "active_s", "true", "other_s", "bar"));
+ assertU(adoc("id", "3", f, "30", "active_s", "false", "other_s", "foo"));
+ assertU(adoc("id", "4", f, "40", "active_s", "false", "other_s", "foo"));
assertU(commit());
- assertQ("test value for active_s=true", req("q","*:*", "stats","true", "stats.field",f, "stats.facet","active_s","indent","true")
- , "//lst[@name='true']/double[@name='min'][.='10.0']"
- , "//lst[@name='true']/double[@name='max'][.='20.0']"
- , "//lst[@name='true']/double[@name='sum'][.='30.0']"
- , "//lst[@name='true']/long[@name='count'][.='2']"
- , "//lst[@name='true']/long[@name='missing'][.='0']"
- , "//lst[@name='true']/double[@name='sumOfSquares'][.='500.0']"
- , "//lst[@name='true']/double[@name='mean'][.='15.0']"
- , "//lst[@name='true']/double[@name='stddev'][.='7.0710678118654755']"
+ final String pre = "//lst[@name='stats_fields']/lst[@name='"+f+"']/lst[@name='facets']/lst[@name='active_s']";
+
+ assertQ("test value for active_s=true", req("q","*:*", "stats","true", "stats.field",f, "stats.facet","active_s","stats.facet","other_s","indent","true")
+ , "*[count("+pre+")=1]"
+ , pre+"/lst[@name='true']/double[@name='min'][.='10.0']"
+ , pre+"/lst[@name='true']/double[@name='max'][.='20.0']"
+ , pre+"/lst[@name='true']/double[@name='sum'][.='30.0']"
+ , pre+"/lst[@name='true']/long[@name='count'][.='2']"
+ , pre+"/lst[@name='true']/long[@name='missing'][.='0']"
+ , pre+"/lst[@name='true']/double[@name='sumOfSquares'][.='500.0']"
+ , pre+"/lst[@name='true']/double[@name='mean'][.='15.0']"
+ , pre+"/lst[@name='true']/double[@name='stddev'][.='7.0710678118654755']"
);
assertQ("test value for active_s=false", req("q","*:*", "stats","true", "stats.field",f, "stats.facet","active_s")
- , "//lst[@name='false']/double[@name='min'][.='30.0']"
- , "//lst[@name='false']/double[@name='max'][.='40.0']"
- , "//lst[@name='false']/double[@name='sum'][.='70.0']"
- , "//lst[@name='false']/long[@name='count'][.='2']"
- , "//lst[@name='false']/long[@name='missing'][.='0']"
- , "//lst[@name='false']/double[@name='sumOfSquares'][.='2500.0']"
- , "//lst[@name='false']/double[@name='mean'][.='35.0']"
- , "//lst[@name='false']/double[@name='stddev'][.='7.0710678118654755']"
+ , pre+"/lst[@name='false']/double[@name='min'][.='30.0']"
+ , pre+"/lst[@name='false']/double[@name='max'][.='40.0']"
+ , pre+"/lst[@name='false']/double[@name='sum'][.='70.0']"
+ , pre+"/lst[@name='false']/long[@name='count'][.='2']"
+ , pre+"/lst[@name='false']/long[@name='missing'][.='0']"
+ , pre+"/lst[@name='false']/double[@name='sumOfSquares'][.='2500.0']"
+ , pre+"/lst[@name='false']/double[@name='mean'][.='35.0']"
+ , pre+"/lst[@name='false']/double[@name='stddev'][.='7.0710678118654755']"
);
}