You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by yo...@apache.org on 2009/08/05 14:44:38 UTC
svn commit: r801189 -
/lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java
Author: yonik
Date: Wed Aug 5 12:44:38 2009
New Revision: 801189
URL: http://svn.apache.org/viewvc?rev=801189&view=rev
Log:
remove unused code from date faceting
Modified:
lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java
Modified: lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java?rev=801189&r1=801188&r2=801189&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/request/SimpleFacets.java Wed Aug 5 12:44:38 2009
@@ -595,8 +595,6 @@
Date low = start;
while (low.before(end)) {
dmp.setNow(low);
- final String lowI = ft.toInternal(low);
- // final String label = ft.indexedToReadable(lowI);
String label = ft.toExternal(low);
Date high = dmp.parseMath(gap);
@@ -612,8 +610,6 @@
(SolrException.ErrorCode.BAD_REQUEST,
"date facet infinite loop (is gap negative?)");
}
- final String highI = ft.toInternal(high);
- // resInner.add(label, rangeCount(sf,lowI,highI,true,true));
resInner.add(label, rangeCount(sf,low,high,true,true));
low = high;
}
@@ -638,10 +634,7 @@
// no matter what other values are listed, we don't do
// anything if "none" is specified.
- if (! others.contains(FacetDateOther.NONE) ) {
- final String startI = ft.toInternal(start);
- final String endI = ft.toInternal(end);
-
+ if (! others.contains(FacetDateOther.NONE) ) {
boolean all = others.contains(FacetDateOther.ALL);
if (all || others.contains(FacetDateOther.BEFORE)) {