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 945ee824120 (Fix)(nereids) modify create view privilege check error
message (#33669)
945ee824120 is described below
commit 945ee82412051510dcce17dbe559ba8e77bc0231
Author: feiniaofeiafei <[email protected]>
AuthorDate: Tue Apr 16 19:09:06 2024 +0800
(Fix)(nereids) modify create view privilege check error message (#33669)
---
.../doris/nereids/trees/plans/commands/info/CreateViewInfo.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CreateViewInfo.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CreateViewInfo.java
index 6edca0a9877..084fe3af3f4 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CreateViewInfo.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CreateViewInfo.java
@@ -106,8 +106,6 @@ public class CreateViewInfo {
/**validate*/
public void validate(ConnectContext ctx) throws UserException {
- NereidsPlanner planner = new NereidsPlanner(ctx.getStatementContext());
- planner.plan(new UnboundResultSink<>(logicalQuery),
PhysicalProperties.ANY, ExplainLevel.NONE);
viewName.analyze(ctx);
FeNameFormat.checkTableName(viewName.getTbl());
// disallow external catalog
@@ -115,8 +113,11 @@ public class CreateViewInfo {
// check privilege
if (!Env.getCurrentEnv().getAccessManager().checkTblPriv(ctx, new
TableName(viewName.getCtl(), viewName.getDb(),
viewName.getTbl()), PrivPredicate.CREATE)) {
-
ErrorReport.reportAnalysisException(ErrorCode.ERR_SPECIFIC_ACCESS_DENIED_ERROR,
"CREATE");
+
ErrorReport.reportAnalysisException(ErrorCode.ERR_TABLE_ACCESS_DENIED_ERROR,
+ PrivPredicate.CREATE.getPrivs().toString(),
viewName.getTbl());
}
+ NereidsPlanner planner = new NereidsPlanner(ctx.getStatementContext());
+ planner.plan(new UnboundResultSink<>(logicalQuery),
PhysicalProperties.ANY, ExplainLevel.NONE);
Set<String> colSets = Sets.newTreeSet(String.CASE_INSENSITIVE_ORDER);
for (Column col : finalCols) {
if (!colSets.add(col.getName())) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
