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 3918b324891230289d5743d5410674b22b983dd2 Author: woyumen4597 <[email protected]> AuthorDate: Sat Feb 1 13:44:12 2020 +0800 KYLIN-4364 minor,limit varchar length to DefaultVarcharPrecision --- source-jdbc/src/main/java/org/apache/kylin/source/jdbc/JdbcExplorer.java | 1 + 1 file changed, 1 insertion(+) diff --git a/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/JdbcExplorer.java b/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/JdbcExplorer.java index 9b6d53e..df5fe0c 100644 --- a/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/JdbcExplorer.java +++ b/source-jdbc/src/main/java/org/apache/kylin/source/jdbc/JdbcExplorer.java @@ -301,6 +301,7 @@ public class JdbcExplorer implements ISourceMetadataExplorer, ISampleDataDeploye String kylinType = SqlUtil.jdbcTypeToKylinDataType(type); 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());
