macroguo-ghy commented on code in PR #38680:
URL: https://github.com/apache/doris/pull/38680#discussion_r1699559515
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java:
##########
@@ -3009,10 +3012,23 @@ protected LogicalPlan withSelectQuerySpecification(
throw new ParseException("only column name is
supported in except clause", selectColumnCtx);
}
UnboundStar star = new UnboundStar(ImmutableList.of());
- project = new LogicalProject<>(ImmutableList.of(star),
expressions, isDistinct, aggregate);
+ project = new LogicalProject<>(
+ ImmutableList.of(star), expressions,
ImmutableList.of(), isDistinct, aggregate);
+ } else if (selectColumnCtx.REPLACE() != null) {
Review Comment:
ok, I will fix it. I think we should support combine `REPLACE` and `EXCEPT`.
I see ck support this syntax,
https://clickhouse.com/docs/en/sql-reference/statements/select#modifier-combinations
--
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.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
