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 b27667efff88eb51ed35a72e5c595ebed77b21d5
Author: woyumen4597 <[email protected]>
AuthorDate: Sun Feb 2 12:56:04 2020 +0800

    minor,add check to extensible/JdbcExplorer
---
 .../main/java/org/apache/kylin/source/jdbc/extensible/JdbcExplorer.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/extensible/JdbcExplorer.java
 
b/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/extensible/JdbcExplorer.java
index e9330e1..06e388f 100644
--- 
a/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/extensible/JdbcExplorer.java
+++ 
b/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/extensible/JdbcExplorer.java
@@ -19,6 +19,7 @@ package org.apache.kylin.source.jdbc.extensible;
 
 import com.google.common.collect.Maps;
 import org.apache.commons.lang3.StringUtils;
+import org.apache.kylin.common.KylinConfig;
 import org.apache.kylin.common.util.Pair;
 import org.apache.kylin.metadata.datatype.DataType;
 import org.apache.kylin.metadata.model.ColumnDesc;
@@ -107,6 +108,7 @@ public class JdbcExplorer implements 
ISourceMetadataExplorer, ISampleDataDeploye
                     kylinType = dataSource.toKylinTypeName(kylinTypeId);
                 }
                 int precision = (SqlUtil.isPrecisionApplicable(kylinType) && 
csize > 0) ? csize : -1;
+                precision = Math.min(precision, 
KylinConfig.getInstanceFromEnv().getDefaultVarcharPrecision());
                 int scale = (SqlUtil.isScaleApplicable(kylinType) && digits > 
0) ? digits : -1;
 
                 cdesc.setDatatype(new DataType(kylinType, precision, 
scale).toString());

Reply via email to