Repository: kylin
Updated Branches:
  refs/heads/yang22-hbase102 9c093f78a -> aba62e894 (forced update)


KYLIN-2382 fix column order for select *


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/8bfef648
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/8bfef648
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/8bfef648

Branch: refs/heads/yang22-hbase102
Commit: 8bfef648956fdc56031477016fd24628eddd0bea
Parents: 2e08815
Author: Yang Li <liy...@apache.org>
Authored: Thu Jan 12 08:08:29 2017 +0800
Committer: Yang Li <liy...@apache.org>
Committed: Thu Jan 12 08:15:47 2017 +0800

----------------------------------------------------------------------
 .../main/java/org/apache/kylin/query/schema/OLAPTable.java   | 8 ++++++++
 1 file changed, 8 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/8bfef648/query/src/main/java/org/apache/kylin/query/schema/OLAPTable.java
----------------------------------------------------------------------
diff --git a/query/src/main/java/org/apache/kylin/query/schema/OLAPTable.java 
b/query/src/main/java/org/apache/kylin/query/schema/OLAPTable.java
index 06ffdba..b18b90d 100644
--- a/query/src/main/java/org/apache/kylin/query/schema/OLAPTable.java
+++ b/query/src/main/java/org/apache/kylin/query/schema/OLAPTable.java
@@ -19,6 +19,8 @@
 package org.apache.kylin.query.schema;
 
 import java.util.ArrayList;
+import java.util.Collections;
+import java.util.Comparator;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
@@ -219,6 +221,12 @@ public class OLAPTable extends AbstractQueryableTable 
implements TranslatableTab
             logger.info("To avoid overflow, upgraded {}'s type from {} to {}", 
tableColumns.get(index), tableColumns.get(index).getType(), 
tableColumns.get(index).getUpgradedType());
         }
 
+        Collections.sort(tableColumns, new Comparator<ColumnDesc>() {
+            @Override
+            public int compare(ColumnDesc o1, ColumnDesc o2) {
+                return o1.getZeroBasedIndex() - o2.getZeroBasedIndex();
+            }
+        });
         return Lists.newArrayList(Iterables.concat(tableColumns, 
metricColumns));
     }
 

Reply via email to