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:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to