diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java index 3af445884a7..c75e638f0d3 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java @@ -2806,13 +2806,6 @@ public class MasterRpcServices extends RSRpcServices mergeExistingPermissions); } master.cpHost.postGrant(perm, mergeExistingPermissions); - User caller = RpcServer.getRequestUser().orElse(null); - if (AUDITLOG.isTraceEnabled()) { - // audit log should store permission changes in addition to auth results - String remoteAddress = RpcServer.getRemoteAddress().map(InetAddress::toString).orElse(""); - AUDITLOG.trace("User {} (remote address: {}) granted permission {}", caller, - remoteAddress, perm); - } return GrantResponse.getDefaultInstance(); } else { throw new DoNotRetryIOException( @@ -2836,13 +2829,6 @@ public class MasterRpcServices extends RSRpcServices PermissionStorage.removeUserPermission(master.getConfiguration(), userPermission, table); } master.cpHost.postRevoke(userPermission); - User caller = RpcServer.getRequestUser().orElse(null); - if (AUDITLOG.isTraceEnabled()) { - // audit log should record all permission changes - String remoteAddress = RpcServer.getRemoteAddress().map(InetAddress::toString).orElse(""); - AUDITLOG.trace("User {} (remote address: {}) revoked permission {}", caller, - remoteAddress, userPermission); - } return RevokeResponse.getDefaultInstance(); } else { throw new DoNotRetryIOException(