You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2018/04/04 15:33:44 UTC

[kylin] branch master updated: Revert "KYLIN-3296 Fix the split rowkey problem"

This is an automated email from the ASF dual-hosted git repository.

shaofengshi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/master by this push:
     new 1e884f2  Revert "KYLIN-3296 Fix the split rowkey problem"
1e884f2 is described below

commit 1e884f248602efa80cff0009dad73b94098363ff
Author: shaofengshi <sh...@apache.org>
AuthorDate: Wed Apr 4 23:33:24 2018 +0800

    Revert "KYLIN-3296 Fix the split rowkey problem"
    
    This reverts commit be3588b7f177d6621a09fefe3ad458499ceac97d.
---
 .../src/main/java/org/apache/kylin/cube/common/RowKeySplitter.java      | 2 --
 1 file changed, 2 deletions(-)

diff --git a/core-cube/src/main/java/org/apache/kylin/cube/common/RowKeySplitter.java b/core-cube/src/main/java/org/apache/kylin/cube/common/RowKeySplitter.java
index 9286fba..0c54ecf 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/common/RowKeySplitter.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/common/RowKeySplitter.java
@@ -124,8 +124,6 @@ public class RowKeySplitter implements java.io.Serializable {
             int colLength = colIO.getColumnLength(col);
             SplittedBytes split = this.splitBuffers[this.bufferSize++];
             split.length = colLength;
-            if(split.length > split.value.length)
-                split.value = new byte[split.length];
             System.arraycopy(bytes, offset, split.value, 0, colLength);
             offset += colLength;
         }

-- 
To stop receiving notification emails like this one, please contact
shaofengshi@apache.org.