You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/08/08 17:05:29 UTC

[GitHub] [accumulo] ctubbsii commented on pull request #2814: Adding ChangeSecret utility to the accumulo admin command

ctubbsii commented on PR #2814:
URL: https://github.com/apache/accumulo/pull/2814#issuecomment-1208381006

   > Changes look good and I confirmed the command works as intended. My only question is if we intend to keep the use of SiteConfiguration or not (see issue #2155).
   
   It is fine to use SiteConfiguration. The Admin command is a server-side utility only, requires the server base jar. The shell is a client side utility and should not depend on the server jars for its class path. That's the difference.
   
   I haven't looked at the rest of the PR, but wanted to address that comment to clarify.


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