From e9064d1b947d86a1d825109a2b505ae94def3403 Mon Sep 17 00:00:00 2001 From: morrySnow <101034200+morrySnow@users.noreply.github.com> Date: Mon, 29 Apr 2024 19:26:13 +0800 Subject: [PATCH] [fix](Nereids) topn should not inherit logical properties when repace child (#34282) --- .../doris/nereids/trees/plans/logical/LogicalTopN.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalTopN.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalTopN.java index dd1c171ca2..6def25d242 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalTopN.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalTopN.java @@ -137,16 +137,11 @@ public class LogicalTopN extends LogicalUnary withLimitChild(long limit, long offset, Plan child) { - Preconditions.checkArgument(children.size() == 1, - "LogicalTopN should have 1 child, but input is %s", children.size()); return new LogicalTopN<>(orderKeys, limit, offset, child); } public LogicalTopN withLimitOrderKeyAndChild(long limit, long offset, List orderKeys, Plan child) { - Preconditions.checkArgument(children.size() == 1, - "LogicalTopN should have 1 child, but input is %s", children.size()); - return new LogicalTopN<>(orderKeys, limit, offset, - Optional.empty(), Optional.of(getLogicalProperties()), child); + return new LogicalTopN<>(orderKeys, limit, offset, child); } @Override