From da4ffd3c5667b26901cabbda07b9034c716237a9 Mon Sep 17 00:00:00 2001 From: camby <104178625@qq.com> Date: Wed, 31 Aug 2022 14:35:48 +0800 Subject: [PATCH] [Enhancement](metric-type) more readable error message for only metric type #12162 Co-authored-by: cambyzju --- .../org/apache/doris/analysis/SetOperationStmt.java | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/SetOperationStmt.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/SetOperationStmt.java index 7fdd67c8d6..b3ef8269e6 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/analysis/SetOperationStmt.java +++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/SetOperationStmt.java @@ -234,7 +234,7 @@ public class SetOperationStmt extends QueryStmt { * set operands are set compatible, adding implicit casts if necessary. */ @Override - public void analyze(Analyzer analyzer) throws UserException { + public void analyze(Analyzer analyzer) throws AnalysisException, UserException { if (isAnalyzed()) { return; } @@ -292,13 +292,8 @@ public class SetOperationStmt extends QueryStmt { if (!distinctOperands.isEmpty()) { // Aggregate produces exactly the same tuple as the original setOp stmt. ArrayList groupingExprs = Expr.cloneList(resultExprs); - try { - distinctAggInfo = AggregateInfo.create( - groupingExprs, null, analyzer.getDescTbl().getTupleDesc(tupleId), analyzer); - } catch (AnalysisException e) { - // Should never happen. - throw new IllegalStateException("Error creating agg info in SetOperationStmt.analyze()", e); - } + distinctAggInfo = AggregateInfo.create( + groupingExprs, null, analyzer.getDescTbl().getTupleDesc(tupleId), analyzer); } setOpsResultExprs = Expr.cloneList(resultExprs);