You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by yo...@apache.org on 2010/11/15 02:30:54 UTC
svn commit: r1035117 -
/lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java
Author: yonik
Date: Mon Nov 15 01:30:54 2010
New Revision: 1035117
URL: http://svn.apache.org/viewvc?rev=1035117&view=rev
Log:
SOLR-236: grouping - remove redundant setBottom
Modified:
lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java
Modified: lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java?rev=1035117&r1=1035116&r2=1035117&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java (original)
+++ lucene/dev/trunk/solr/src/java/org/apache/solr/search/Grouping.java Mon Nov 15 01:30:54 2010
@@ -715,10 +715,9 @@ class TopGroupSortCollector extends TopG
groupMap.put(smallest.groupValue, smallest);
orderedGroups.add(smallest);
+ int lastSlot = orderedGroups.last().comparatorSlot;
for (FieldComparator fc : comparators)
- fc.setBottom(orderedGroups.last().comparatorSlot);
- for (FieldComparator fc : smallest.sortGroupComparators)
- fc.setBottom(0);
+ fc.setBottom(lastSlot);
return;
}