You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ja...@apache.org on 2023/04/22 04:33:01 UTC
[doris] branch master updated: [feat](Nereids): validate Project list. (#18868)
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 <ja...@gmail.com>
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: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org