[GitHub] [accumulo] dlmarion commented on a change in pull request #2415: Create new external compactions admin utility

[GitBox <gi...@apache.org>]

dlmarion commented on a change in pull request #2415:
URL: https://github.com/apache/accumulo/pull/2415#discussion_r786715603



##########
File path: server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
##########
@@ -360,6 +360,12 @@ private void cancel(String externalCompactionId) throws TException {
     }
   }
 
+  @Override
+  public void cancel(TInfo tinfo, TCredentials credentials, String externalCompactionId)
+      throws TException {

Review comment:
       The other thrift api methods perform a security check first, do we want to do that here?




-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org