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

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

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



##########
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:
       I didn't think it needed SYSTEM level permission (root) like the other method required. I thought it should require at least some level of USER permission but wasn't sure since the permissions are so confusing. Any ideas?




-- 
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