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

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

commit 7886a2498178ded085497d6204b4b367f56ce0c8
Author: Zhong, Yanghong <nju_y...@apache.org>
AuthorDate: Tue Mar 10 16:44:15 2020 +0800

    KYLIN-4418 Bug fix for ShardingHash.getShard in HBaseLookupRowEncoder
---
 .../org/apache/kylin/storage/hbase/lookup/HBaseLookupRowEncoder.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/lookup/HBaseLookupRowEncoder.java
 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/lookup/HBaseLookupRowEncoder.java
index 7269465..51bd32b 100644
--- 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/lookup/HBaseLookupRowEncoder.java
+++ 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/lookup/HBaseLookupRowEncoder.java
@@ -88,7 +88,8 @@ public class HBaseLookupRowEncoder extends 
AbstractLookupRowEncoder<HBaseRow> {
         }
         byte[] result = new byte[RowConstants.ROWKEY_SHARDID_LEN + 
keyByteBuffer.position()];
         System.arraycopy(keyByteBuffer.array(), 0, result, 
RowConstants.ROWKEY_SHARDID_LEN, keyByteBuffer.position());
-        short shard = ShardingHash.getShard(result, 
RowConstants.ROWKEY_SHARDID_LEN, result.length, shardNum);
+        short shard = ShardingHash.getShard(result, 
RowConstants.ROWKEY_SHARDID_LEN,
+                result.length - RowConstants.ROWKEY_SHARDID_LEN, shardNum);
         BytesUtil.writeShort(shard, result, 0, 
RowConstants.ROWKEY_SHARDID_LEN);
         return result;
     }

Reply via email to