column privilege

924060929 · Mar 3, 2024 · 68e1d3d · 68e1d3d
1 parent 3f63285
commit 68e1d3d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/...rc/main/java/org/apache/doris/nereids/rules/rewrite/EliminateSortUnderSubqueryOrView.java b/...rc/main/java/org/apache/doris/nereids/rules/rewrite/EliminateSortUnderSubqueryOrView.java
@@ -35,7 +35,7 @@ public List<Rule> buildRules() {
                 logicalSubQueryAlias(logicalSort())
                         .then(subq -> subq.withChildren(subq.child().child(0)))
             ),
-            RuleType.ELIMINATE_ORDER_BY_UNDER_SUBQUERY.build(
+            RuleType.ELIMINATE_ORDER_BY_UNDER_VIEW.build(
                 logicalView(logicalSort())
                         .then(view -> view.withChildren(view.child().child(0)))
             )

diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalView.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalView.java
@@ -18,6 +18,7 @@
 package org.apache.doris.nereids.trees.plans.logical;
 
 import org.apache.doris.catalog.View;
+import org.apache.doris.nereids.exceptions.AnalysisException;
 import org.apache.doris.nereids.memo.GroupExpression;
 import org.apache.doris.nereids.properties.FdItem;
 import org.apache.doris.nereids.properties.FunctionalDependencies;
@@ -29,7 +30,6 @@
 import org.apache.doris.nereids.trees.plans.visitor.PlanVisitor;
 import org.apache.doris.nereids.util.Utils;
 
-import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 
@@ -46,7 +46,9 @@ public class LogicalView<BODY extends Plan> extends LogicalUnary<BODY> {
     public LogicalView(View view, BODY body) {
         super(PlanType.LOGICAL_VIEW, Optional.empty(), Optional.empty(), body);
         this.view = Objects.requireNonNull(view, "catalog can not be null");
-        Preconditions.checkArgument(body instanceof LogicalPlan);
+        if (!(body instanceof LogicalPlan)) {
+            throw new AnalysisException("Child of LogicalView should be LogicalPlan, but meet: " + body.getClass());
+        }
     }
 
     @Override