[kylin] branch master updated: KYLIN-4020 fix_length rowkey encode without sepecified length can be saved but cause CreateHTable step failed

[ni...@apache.org]

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


The following commit(s) were added to refs/heads/master by this push:
     new 89aa466  KYLIN-4020 fix_length rowkey encode without sepecified length can be saved but cause CreateHTable step failed
89aa466 is described below

commit 89aa466197fcc2105af05775e779eb771fbd607e
Author: yuzhang <sh...@163.com>
AuthorDate: Thu May 30 01:34:17 2019 +0800

    KYLIN-4020 fix_length rowkey encode without sepecified length can be saved but cause CreateHTable step failed
---
 webapp/app/js/controllers/cubeSchema.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/webapp/app/js/controllers/cubeSchema.js b/webapp/app/js/controllers/cubeSchema.js
index 31a5afe..582873f 100755
--- a/webapp/app/js/controllers/cubeSchema.js
+++ b/webapp/app/js/controllers/cubeSchema.js
@@ -382,6 +382,9 @@ KylinApp.controller('CubeSchemaCtrl', function ($scope, QueryService, UserServic
           if(rowkey.encoding.substr(0,3)=='int' && (rowkey.encoding.substr(4)<1 || rowkey.encoding.substr(4)>8)){
             errors.push("int encoding column length should between 1 and 8.");
           }
+          if(rowkey.encoding.substr(0, 5) == 'fixed' && (!/^[1-9]\d*$/.test(rowkey.encoding.split(':')[1]))) {
+            errors.push("fixed encoding need a valid length.")
+          }
         })
         if(shardRowkeyList.length >1){
           errors.push("At most one 'shard by' column is allowed.");