This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a commit to branch branch-2.1 in repository https://gitbox.apache.org/repos/asf/doris.git
commit 2b3e7589b74fd5657012378a009479d15005ec40 Author: starocean999 <[email protected]> AuthorDate: Tue Jan 23 19:38:27 2024 +0800 [fix](nereids)group by expr may be lost in EliminateGroupByConstant rule (#30274) --- .../apache/doris/nereids/rules/analysis/EliminateGroupByConstant.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/EliminateGroupByConstant.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/EliminateGroupByConstant.java index e7fa14e5cb2..e683153e9a2 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/EliminateGroupByConstant.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/EliminateGroupByConstant.java @@ -67,7 +67,7 @@ public class EliminateGroupByConstant extends OneRewriteRuleFactory { lit = expression; } } - if (slotGroupByExprs.isEmpty() && lit != null && aggregate.getAggregateFunctions().isEmpty()) { + if (slotGroupByExprs.isEmpty() && lit != null) { slotGroupByExprs.add(lit); } return aggregate.withGroupByAndOutput(ImmutableList.copyOf(slotGroupByExprs), outputExprs); --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
