From ecfa6bd1b16a9e2a5a80cf4dd3ff1146c48353c2 Mon Sep 17 00:00:00 2001
From: nishantmonu51 <nishant.monu51@gmail.com>
Date: Tue, 8 Apr 2014 17:26:51 +0530
Subject: [PATCH] fix casing and add comment

---
 processing/src/main/java/io/druid/query/topn/TopNBinaryFn.java | 3 ++-
 .../java/io/druid/query/topn/TopNNumericResultBuilder.java     | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/processing/src/main/java/io/druid/query/topn/TopNBinaryFn.java b/processing/src/main/java/io/druid/query/topn/TopNBinaryFn.java
index f848f3c9e06..8c91b8f7b53 100644
--- a/processing/src/main/java/io/druid/query/topn/TopNBinaryFn.java
+++ b/processing/src/main/java/io/druid/query/topn/TopNBinaryFn.java
@@ -94,6 +94,7 @@ public class TopNBinaryFn implements BinaryFn<Result<TopNResultValue>, Result<To
       DimensionAndMetricValueExtractor arg1Val = retVals.get(dimensionValue);
 
       if (arg1Val != null) {
+        // size of map = aggregattor + topNDim + postAgg (If sorting is done on post agg field)
         Map<String, Object> retVal = new LinkedHashMap<String, Object>(aggregations.size() + 2);
 
         retVal.put(dimension, dimensionValue);
@@ -102,7 +103,7 @@ public class TopNBinaryFn implements BinaryFn<Result<TopNResultValue>, Result<To
           retVal.put(metricName, factory.combine(arg1Val.getMetric(metricName), arg2Val.getMetric(metricName)));
         }
         for (PostAggregator postAgg : postAggregations) {
-          if (postAgg.getName().equals(topNMetricName)) {
+          if (postAgg.getName().equalsIgnoreCase(topNMetricName)) {
             retVal.put(postAgg.getName(), postAgg.compute(retVal));
           }
         }
diff --git a/processing/src/main/java/io/druid/query/topn/TopNNumericResultBuilder.java b/processing/src/main/java/io/druid/query/topn/TopNNumericResultBuilder.java
index a806c7966ec..bfe142efbb5 100644
--- a/processing/src/main/java/io/druid/query/topn/TopNNumericResultBuilder.java
+++ b/processing/src/main/java/io/druid/query/topn/TopNNumericResultBuilder.java
@@ -76,7 +76,7 @@ public class TopNNumericResultBuilder implements TopNResultBuilder
     }
 
     for (PostAggregator postAgg : postAggs) {
-      if (postAgg.getName().equals(metricName)) {
+      if (postAgg.getName().equalsIgnoreCase(metricName)) {
         metricValues.put(postAgg.getName(), postAgg.compute(metricValues));
         break;
       }