From 642897dca3aabcd38f0aec1047ba593599cf03da Mon Sep 17 00:00:00 2001
From: fjy <fangjin@metamarkets.com>
Date: Wed, 28 May 2014 10:43:03 -0700
Subject: [PATCH] fix inverted logic

---
 .../java/io/druid/query/dimension/DefaultDimensionSpec.java     | 2 +-
 .../src/main/java/io/druid/query/topn/PooledTopNAlgorithm.java  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/processing/src/main/java/io/druid/query/dimension/DefaultDimensionSpec.java b/processing/src/main/java/io/druid/query/dimension/DefaultDimensionSpec.java
index 78fb2b4d16..ee5ee67429 100644
--- a/processing/src/main/java/io/druid/query/dimension/DefaultDimensionSpec.java
+++ b/processing/src/main/java/io/druid/query/dimension/DefaultDimensionSpec.java
@@ -79,7 +79,7 @@ public class DefaultDimensionSpec implements DimensionSpec
   @Override
   public boolean preservesOrdering()
   {
-    return false;
+    return true;
   }
 
   @Override
diff --git a/processing/src/main/java/io/druid/query/topn/PooledTopNAlgorithm.java b/processing/src/main/java/io/druid/query/topn/PooledTopNAlgorithm.java
index 437b056da6..ea4fd37547 100644
--- a/processing/src/main/java/io/druid/query/topn/PooledTopNAlgorithm.java
+++ b/processing/src/main/java/io/druid/query/topn/PooledTopNAlgorithm.java
@@ -116,7 +116,7 @@ public class PooledTopNAlgorithm
   {
     final TopNMetricSpecBuilder<int[]> arrayProvider = params.getArrayProvider();
 
-    if (query.getDimensionSpec().preservesOrdering()) {
+    if (!query.getDimensionSpec().preservesOrdering()) {
       return arrayProvider.build();
     }
 
-- 
GitLab