morningman commented on a change in pull request #3135: support subquery in 
case when statement
URL: https://github.com/apache/incubator-doris/pull/3135#discussion_r393733325
 
 

 ##########
 File path: fe/src/main/java/org/apache/doris/analysis/SelectStmt.java
 ##########
 @@ -1244,6 +1213,46 @@ public void rewriteExprs(ExprRewriter rewriter) throws 
AnalysisException {
                 orderByElem.setExpr(rewriter.rewrite(orderByElem.getExpr(), 
analyzer));
             }
         }
+        if (subqueryInCase) {
+            for (SelectListItem item : selectList.getItems()) {
+                if (!(item.getExpr() instanceof CaseExpr)) {
+                    continue;
+                }
+                if 
(!item.getExpr().contains(Predicates.instanceOf(Subquery.class))) {
+                    continue;
+                }
+                item.setExpr(rewriteSubquery(item.getExpr()));
+            }
+        }
+    }
+
+
+    private Expr rewriteSubquery(Expr expr) throws AnalysisException {
+        if (expr instanceof Subquery) {
+            if (!(((Subquery) expr).getStatement() instanceof SelectStmt) ) {
+                throw new AnalysisException("only support select subquery in 
case statement.");
+            }
+            SelectStmt subquery = (SelectStmt)((Subquery) expr).getStatement();
+            if (subquery.getSelectList().getItems().size() != 1) {
 
 Review comment:
   How do you make sure the subquery is like `select sum(k1) from tbl`, not 
`select k1 from tbl`?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to