ctubbsii commented on code in PR #2215:
URL: https://github.com/apache/accumulo/pull/2215#discussion_r878582034
##########
server/base/src/main/java/org/apache/accumulo/server/client/ClientServiceHandler.java:
##########
@@ -447,6 +460,83 @@ public List<TDiskUsage> getDiskUsage(Set<String> tables,
TCredentials credential
}
}
+ @Override
+ public List<FateTransaction> executeAdminOperation(TInfo tInfo, TCredentials
credentials,
+ AdminOperation op, List<String> txids, List<String> filterStatuses)
+ throws ThriftSecurityException, TException {
+ try {
+ authenticate(tInfo, credentials);
Review Comment:
> What do you mean by "admin permissions" here? It looks like this checks
that the credentials contain `SYSTEM` permissions.
What line do you see that?
--
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]
