This is an automated email from the ASF dual-hosted git repository.

morrysnow 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 96518db263 [enhencement](Nereids) remove constant expr constraint on 
OneRowRelation (#15506)
96518db263 is described below

commit 96518db2631109c7f4f9450375599e908d9205a5
Author: jakevin <[email protected]>
AuthorDate: Fri Dec 30 23:35:15 2022 +0800

    [enhencement](Nereids) remove constant expr constraint on OneRowRelation 
(#15506)
---
 .../nereids/trees/plans/physical/PhysicalOneRowRelation.java     | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOneRowRelation.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOneRowRelation.java
index 02885fee30..ed59190807 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOneRowRelation.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOneRowRelation.java
@@ -29,7 +29,6 @@ import 
org.apache.doris.nereids.trees.plans.visitor.PlanVisitor;
 import org.apache.doris.nereids.util.Utils;
 import org.apache.doris.statistics.StatsDeriveResult;
 
-import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 
 import java.util.List;
@@ -45,19 +44,17 @@ public class PhysicalOneRowRelation extends PhysicalLeaf 
implements OneRowRelati
     private final boolean buildUnionNode;
 
     public PhysicalOneRowRelation(List<NamedExpression> projects, boolean 
buildUnionNode,
-                                  LogicalProperties logicalProperties) {
+            LogicalProperties logicalProperties) {
         this(projects, buildUnionNode, Optional.empty(), logicalProperties, 
null, null);
     }
 
     private PhysicalOneRowRelation(List<NamedExpression> projects,
-                                   boolean buildUnionNode,
-                                   Optional<GroupExpression> groupExpression,
+            boolean buildUnionNode,
+            Optional<GroupExpression> groupExpression,
             LogicalProperties logicalProperties, PhysicalProperties 
physicalProperties,
             StatsDeriveResult statsDeriveResult) {
         super(PlanType.PHYSICAL_ONE_ROW_RELATION, groupExpression, 
logicalProperties, physicalProperties,
                 statsDeriveResult);
-        
Preconditions.checkArgument(projects.stream().allMatch(Expression::isConstant),
-                "OneRowRelation must consist of some constant expression");
         this.projects = ImmutableList.copyOf(Objects.requireNonNull(projects, 
"projects can not be null"));
         this.buildUnionNode = buildUnionNode;
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to