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/10 14:09:02 UTC
kylin git commit: bug fix
Repository: kylin
Updated Branches:
refs/heads/KYLIN-2497-2 47c67f0ab -> 7033f4be5
bug fix
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/7033f4be
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/7033f4be
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/7033f4be
Branch: refs/heads/KYLIN-2497-2
Commit: 7033f4be50a65d8a83dc9a12796c0adf4d39a4ec
Parents: 47c67f0
Author: Yang Li <li...@apache.org>
Authored: Fri Mar 10 22:08:25 2017 +0800
Committer: Yang Li <li...@apache.org>
Committed: Fri Mar 10 22:08:37 2017 +0800
----------------------------------------------------------------------
.../org/apache/kylin/engine/mr/steps/SelfDefineSortableKey.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/7033f4be/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 07ad850..ace753a 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
@@ -48,8 +48,7 @@ public class SelfDefineSortableKey implements WritableComparable<SelfDefineSorta
this.typeId = typeId;
this.rawKey = key;
if (isNumberFamily()) {
- byte[] buf = key.getBytes();
- String valueStr = new String(buf, 1, buf.length - 1);
+ String valueStr = new String(key.getBytes(), 1, key.getLength() - 1);
if (isIntegerFamily()) {
this.keyInObj = Integer.parseInt(valueStr);
} else {