You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2022/12/16 07:01:38 UTC
[kylin] 07/15: KYLIN-5353 fix model saving failure when there are CCs of timestampdiff/timestampadd function
This is an automated email from the ASF dual-hosted git repository.
xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git
commit 466263216e1007156438c719c54feab67c907aff
Author: Dorris Zhang <ru...@kyligence.io>
AuthorDate: Fri Oct 21 15:16:10 2022 +0800
KYLIN-5353 fix model saving failure when there are CCs of timestampdiff/timestampadd function
from 39687
Co-authored-by: Dorris Zhang <ru...@kyligence.io>
---
.../main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
index 6e029b6b4e..808b732f7a 100644
--- a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
+++ b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
@@ -1114,7 +1114,7 @@ public class ModelSemanticHelper extends BasicService {
return computedColumnDescs.stream().map(ccDesc -> {
AtomicBoolean isValidCC = new AtomicBoolean(true);
List<Pair<String, String>> colsWithAlias = ComputedColumnUtil.ExprIdentifierFinder
- .getExprIdentifiers(ccDesc.getInnerExpression());
+ .getExprIdentifiers(ccDesc.getExpression());
colsWithAlias.forEach(c -> {
String column = c.getFirst() + "." + c.getSecond();
if (!aliasDotColSet.contains(column)) {