From 200a93ed97372877264185e1e4ba440da7a07c63 Mon Sep 17 00:00:00 2001 From: "peng.jianhua" Date: Thu, 12 Oct 2017 09:19:12 +0800 Subject: [PATCH 1/1] KYLIN-2931 Update correct cardinality for the modified table --- .../java/org/apache/kylin/metadata/model/TableExtDesc.java | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/model/TableExtDesc.java b/core-metadata/src/main/java/org/apache/kylin/metadata/model/TableExtDesc.java index 42da32e7e..637e59110 100644 --- a/core-metadata/src/main/java/org/apache/kylin/metadata/model/TableExtDesc.java +++ b/core-metadata/src/main/java/org/apache/kylin/metadata/model/TableExtDesc.java @@ -164,8 +164,16 @@ public class TableExtDesc extends RootPersistentEntity { this.columnStats.get(i).setCardinality(Long.parseLong(cardi[i])); } } else { - throw new IllegalArgumentException("The given cardinality columns don't match tables " + tableIdentity); - + int columnAmount = (this.columnStats.size() > cardi.length) ? cardi.length : this.columnStats.size(); + int i = 0; + for (; i < columnAmount; i++) { + this.columnStats.get(i).setCardinality(Long.parseLong(cardi[i])); + } + for (; i < cardi.length; i++) { + ColumnStats columnStat = new ColumnStats(); + columnStat.setCardinality(Long.parseLong(cardi[i])); + this.columnStats.add(columnStat); + } } } -- 2.11.0.windows.1