Copilot commented on code in PR #5483:
URL: https://github.com/apache/accumulo/pull/5483#discussion_r2045769780
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletClientHandler.java:
##########
@@ -1146,11 +1142,19 @@ static void checkPermission(SecurityOperation security,
ServerContext context,
throw e;
}
+ if (!canPerformSystemActions) {
+ log.warn("Got {} message from user: {}", request,
credentials.getPrincipal());
Review Comment:
Consider enhancing the log message to clearly indicate that the warning is
raised due to insufficient system action permissions, in order to reduce
potential confusion with other permission errors.
```suggestion
log.warn("Insufficient permissions to perform system actions. Got {}
message from user: {}", request, credentials.getPrincipal());
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
