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 814f12981d [feat](Nereids): validate Project list. (#18868)
814f12981d is described below
commit 814f12981de55162460fe06f99577bb4a2daa414
Author: jakevin <[email protected]>
AuthorDate: Sat Apr 22 12:32:51 2023 +0800
[feat](Nereids): validate Project list. (#18868)
---
.../main/java/org/apache/doris/nereids/processor/post/Validator.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/Validator.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/Validator.java
index 2f86655996..ea24e0183b 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/Validator.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/Validator.java
@@ -42,6 +42,8 @@ public class Validator extends PlanPostProcessor {
@Override
public Plan visitPhysicalProject(PhysicalProject<? extends Plan> project,
CascadesContext context) {
+ Preconditions.checkArgument(!project.getProjects().isEmpty(), "Project
list can't be empty");
+
Plan child = project.child();
// Forbidden project-project, we must merge project.
if (child instanceof PhysicalProject) {
@@ -54,7 +56,7 @@ public class Validator extends PlanPostProcessor {
@Override
public Plan visitPhysicalFilter(PhysicalFilter<? extends Plan> filter,
CascadesContext context) {
Preconditions.checkArgument(!filter.getConjuncts().isEmpty()
- && filter.getPredicate() != BooleanLiteral.TRUE);
+ && filter.getPredicate() != BooleanLiteral.TRUE, "Filter
predicate can't be empty or true");
Plan child = filter.child();
// Forbidden filter-project, we must make filter-project ->
project-filter.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
