You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2017/03/11 08:57:56 UTC

kylin git commit: KYLIN-2497 fix minor bug to support long type

Repository: kylin
Updated Branches:
  refs/heads/yang22 e477b51fe -> 50955963d


KYLIN-2497 fix minor bug to support long type


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

Branch: refs/heads/yang22
Commit: 50955963d117a8a0a05b756cecbb6b1c6d06647d
Parents: e477b51
Author: Hongbin Ma <ma...@apache.org>
Authored: Sat Mar 11 16:58:06 2017 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Sat Mar 11 16:58:06 2017 +0800

----------------------------------------------------------------------
 .../org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/50955963/engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java
----------------------------------------------------------------------
diff --git a/engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java b/engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java
index acec972..bb4152e 100644
--- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java
+++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java
@@ -50,7 +50,7 @@ public class SelfDefineSortableKey implements WritableComparable<SelfDefineSorta
         if (isNumberFamily()) {
             String valueStr = new String(key.getBytes(), 1, key.getLength() - 1);
             if (isIntegerFamily()) {
-                this.keyInObj = Integer.parseInt(valueStr);
+                this.keyInObj = Long.parseLong(valueStr);
             } else {
                 this.keyInObj = Double.parseDouble(valueStr);
             }
@@ -71,7 +71,7 @@ public class SelfDefineSortableKey implements WritableComparable<SelfDefineSorta
             return ((Text) this.keyInObj).compareTo(((Text) o.keyInObj));
         } else {
             if (isIntegerFamily()) {
-                return Integer.compare((Integer) this.keyInObj, (Integer) o.keyInObj);
+                return Long.compare((Long) this.keyInObj, (Long) o.keyInObj);
             } else {
                 return Double.compare((Double) this.keyInObj, (Double) o.keyInObj);
             }