This is an automated email from the ASF dual-hosted git repository.
jakevin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new d774162a53 [minor](Nereids): rename rule (#17509)
d774162a53 is described below
commit d774162a530a636d5076e6bdd497d5de1a871dde
Author: jakevin <[email protected]>
AuthorDate: Sun Mar 12 00:17:07 2023 +0800
[minor](Nereids): rename rule (#17509)
---
.../src/main/java/org/apache/doris/nereids/rules/RuleType.java | 5 ++---
.../apache/doris/nereids/rules/rewrite/logical/PushdownLimit.java | 4 ++--
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java
index f945327630..4f8d96a0d9 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java
@@ -155,6 +155,8 @@ public enum RuleType {
// Eliminate plan
ELIMINATE_AGGREGATE(RuleTypeClass.REWRITE),
ELIMINATE_LIMIT(RuleTypeClass.REWRITE),
+ ELIMINATE_LIMIT_ON_ONE_ROW_RELATION(RuleTypeClass.REWRITE),
+ ELIMINATE_LIMIT_ON_EMPTY_RELATION(RuleTypeClass.REWRITE),
ELIMINATE_FILTER(RuleTypeClass.REWRITE),
ELIMINATE_NOT_NULL(RuleTypeClass.REWRITE),
ELIMINATE_UNNECESSARY_PROJECT(RuleTypeClass.REWRITE),
@@ -195,8 +197,6 @@ public enum RuleType {
PUSH_LIMIT_THROUGH_JOIN(RuleTypeClass.REWRITE),
PUSH_LIMIT_THROUGH_PROJECT_JOIN(RuleTypeClass.REWRITE),
PUSH_LIMIT_THROUGH_UNION(RuleTypeClass.REWRITE),
- PUSH_LIMIT_THROUGH_ONE_ROW_RELATION(RuleTypeClass.REWRITE),
- PUSH_LIMIT_THROUGH_EMPTY_RELATION(RuleTypeClass.REWRITE),
PUSH_LIMIT_INTO_SORT(RuleTypeClass.REWRITE),
// adjust nullable
ADJUST_NULLABLE_ON_AGGREGATE(RuleTypeClass.REWRITE),
@@ -306,6 +306,5 @@ public enum RuleType {
IMPLEMENTATION,
VALIDATION,
SENTINEL,
- ;
}
}
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/PushdownLimit.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/PushdownLimit.java
index 291eda9714..d7f208d36d 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/PushdownLimit.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/PushdownLimit.java
@@ -97,10 +97,10 @@ public class PushdownLimit implements RewriteRuleFactory {
logicalLimit(logicalOneRowRelation())
.then(limit -> limit.getLimit() > 0 &&
limit.getOffset() == 0
? limit.child() : new
LogicalEmptyRelation(limit.child().getOutput()))
- .toRule(RuleType.PUSH_LIMIT_THROUGH_ONE_ROW_RELATION),
+ .toRule(RuleType.ELIMINATE_LIMIT_ON_ONE_ROW_RELATION),
logicalLimit(logicalEmptyRelation())
.then(UnaryNode::child)
- .toRule(RuleType.PUSH_LIMIT_THROUGH_EMPTY_RELATION),
+ .toRule(RuleType.ELIMINATE_LIMIT_ON_EMPTY_RELATION),
new MergeLimits().build()
);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
