Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Removed negating comparator in TopNNumericResultBuilder

This commit is contained in:
Charles Allen 2014-11-07 10:29:06 -08:00
parent ee019872f7
commit 956bdf80dd
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
processing/src/main/java/io/druid/query/topn/TopNNumericResultBuilder.java
View File

@ -181,7 +181,8 @@ public class TopNNumericResultBuilder implements TopNResultBuilder
@Override @Override
public int compare(DimValHolder d1, DimValHolder d2) public int compare(DimValHolder d1, DimValHolder d2)
{ {
int retVal = -metricComparator.compare(d1.getTopNMetricVal(), d2.getTopNMetricVal()); // Values flipped compared to earlier
int retVal = metricComparator.compare(d2.getTopNMetricVal(),d1.getTopNMetricVal());
if (retVal == 0) { if (retVal == 0) {
retVal = dimNameComparator.compare(d1.getDimName(), d2.getDimName()); retVal = dimNameComparator.compare(d1.getDimName(), d2.getDimName());