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());
