From 53889cbbb6f84b50c59069b270372ffe02a8dcc4 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 6 Jan 2025 18:51:09 +0800 Subject: [PATCH] branch-2.1: [enhance](auth)Reduce the scope of permission locks #46115 (#46451) Cherry-picked from #46115 Co-authored-by: zhangdong --- .../main/java/org/apache/doris/mysql/privilege/Auth.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/Auth.java b/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/Auth.java index 3ac1068687..9bca8a2a5d 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/Auth.java +++ b/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/Auth.java @@ -595,11 +595,11 @@ public class Auth implements Writable { private void grantInternal(UserIdentity userIdent, String role, TablePattern tblPattern, PrivBitSet privs, Map> colPrivileges, boolean errOnNonExist, boolean isReplay) throws DdlException { + if (!isReplay) { + checkTablePatternExist(tblPattern, privs); + } writeLock(); try { - if (!isReplay) { - checkTablePatternExist(tblPattern, privs); - } if (role == null) { if (!doesUserExist(userIdent)) { throw new DdlException("user " + userIdent + " does not exist");