seawinde commented on code in PR #26055:
URL: https://github.com/apache/doris/pull/26055#discussion_r1382599382
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindExpression.java:
##########
@@ -564,10 +567,16 @@ protected boolean condition(Rule rule, Plan plan) {
),
RuleType.BINDING_RESULT_SINK.build(
unboundResultSink().then(sink -> {
- List<NamedExpression> outputExprs =
sink.child().getOutput().stream()
- .map(NamedExpression.class::cast)
- .collect(ImmutableList.toImmutableList());
- return new LogicalResultSink<>(outputExprs, sink.child());
+
+ final ImmutableListMultimap.Builder<ExprId, Integer>
exprIdToIndexMapBuilder =
Review Comment:
Ctas stmt is supported in nereids now and it's also call the method
`org.apache.doris.nereids.NereidsPlanner#plan`, this pr is also useful. the old
and new optimizer should keep consistency when query and ddl
--
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
