This is an automated email from the ASF dual-hosted git repository.
morrysnow pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.0 by this push:
new 02885222041 [fix](Nereids) LogicalTopN withLimitChild should not
inherit logical properties (#31480)
02885222041 is described below
commit 02885222041bba4fe573758ea9ea25967ffd9cd8
Author: morrySnow <[email protected]>
AuthorDate: Tue Feb 27 20:03:51 2024 +0800
[fix](Nereids) LogicalTopN withLimitChild should not inherit logical
properties (#31480)
---
.../java/org/apache/doris/nereids/trees/plans/logical/LogicalTopN.java | 3 +--
1 file changed, 1 insertion(+), 2 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 02b239f1735..b1a2f8d1d13 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
@@ -125,8 +125,7 @@ public class LogicalTopN<CHILD_TYPE extends Plan> extends
LogicalUnary<CHILD_TYP
public LogicalTopN<Plan> 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,
- Optional.empty(), Optional.of(getLogicalProperties()), child);
+ return new LogicalTopN<>(orderKeys, limit, offset, child);
}
@Override
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
