[kylin] branch master updated: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)

[bo...@apache.org]

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

boblu 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 7f672b0  KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)
7f672b0 is described below

commit 7f672b074a068d20779d33889f1cb63b0da172a7
Author: Emiya0306 <wo...@qq.com>
AuthorDate: Wed Dec 5 13:24:51 2018 +0800

    KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)
---
 webapp/app/js/controllers/cubeMeasures.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/webapp/app/js/controllers/cubeMeasures.js b/webapp/app/js/controllers/cubeMeasures.js
index 7beb528..f1821dd 100644
--- a/webapp/app/js/controllers/cubeMeasures.js
+++ b/webapp/app/js/controllers/cubeMeasures.js
@@ -465,10 +465,10 @@ KylinApp.controller('CubeMeasuresCtrl', function ($scope, $modal,MetaModel,cubes
           return 'bigint';
         } else {
          if(colType.indexOf('decimal') != -1) {
-            var returnRegex = new RegExp('(\\w+)(?:\\((\\w+?)(?:\\,(\\w+?))?\\))?')
-            var returnValue = returnRegex.exec(colType)
-            var precision = 19
-            var scale = returnValue[3]
+            var returnRegex = new RegExp('(\\w+)(?:\\((\\w+?)(?:\\,(\\w+?))?\\))?');
+            var returnValue = returnRegex.exec(colType);
+            var precision = 19;
+            var scale = returnValue[3] || 0;
             return 'decimal(' + precision + ',' + scale + ')';
           }else{
             return colType;