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 2016/04/22 06:11:31 UTC

[02/50] [abbrv] kylin git commit: Minor bug fix on IntegerDimEnc and test it more in IT

Minor bug fix on IntegerDimEnc and test it more in IT


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

Branch: refs/heads/1.5.x-HBase1.1.3
Commit: 10203c1ce4c4d30a6d09756aa121ba2aa5e814f1
Parents: c7d4449
Author: Hongbin Ma <ma...@apache.org>
Authored: Thu Mar 31 11:24:05 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Thu Mar 31 11:24:21 2016 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java  | 4 ++++
 .../localmeta/cube_desc/test_kylin_cube_with_slr_desc.json       | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/10203c1c/core-metadata/src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java b/core-metadata/src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java
index 15c46a6..1d821c8 100644
--- a/core-metadata/src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java
+++ b/core-metadata/src/main/java/org/apache/kylin/dimension/IntegerDimEnc.java
@@ -56,6 +56,10 @@ public class IntegerDimEnc extends DimensionEncoding {
 
     transient private int avoidVerbose = 0;
 
+    //no-arg constructor is required for Externalizable
+    public IntegerDimEnc() {
+    }
+
     public IntegerDimEnc(int len) {
         if (len <= 0 || len >= CAP.length)
             throw new IllegalArgumentException();

http://git-wip-us.apache.org/repos/asf/kylin/blob/10203c1c/examples/test_case_data/localmeta/cube_desc/test_kylin_cube_with_slr_desc.json
----------------------------------------------------------------------
diff --git a/examples/test_case_data/localmeta/cube_desc/test_kylin_cube_with_slr_desc.json b/examples/test_case_data/localmeta/cube_desc/test_kylin_cube_with_slr_desc.json
index 2253c60..6ac0e82 100644
--- a/examples/test_case_data/localmeta/cube_desc/test_kylin_cube_with_slr_desc.json
+++ b/examples/test_case_data/localmeta/cube_desc/test_kylin_cube_with_slr_desc.json
@@ -113,7 +113,7 @@
   "rowkey" : {
     "rowkey_columns" : [ {
       "column" : "seller_id",
-      "encoding" : "fixed_length:18",
+      "encoding" : "int:4",
       "isShardBy" : true
     }, {
       "column" : "cal_dt",
@@ -123,7 +123,7 @@
       "encoding" : "dict"
     }, {
       "column" : "meta_categ_name",
-      "encoding" : "dict"
+      "encoding" : "fixed_length:18"
     }, {
       "column" : "categ_lvl2_name",
       "encoding" : "dict"