cshannon commented on issue #2807: URL: https://github.com/apache/accumulo/issues/2807#issuecomment-1187567084
@milleruntime - 1. Testing - I ran all the commands against Uno and they _seemed_ to work but since I'm pretty new I wasn't sure how to verify the output on all of them. I looked around and didn't really see any existing unit tests for the utilities themselves to verify they work or functionality. Maybe another task should be to actually write some unit tests for these commands to verify they work (using mini accumulo, etc). I could create a new issue and PRs to add tests (if you agree they should be a separate issue) 2. I agree that I figured some of them would make more sense as part of the Admin command as I mentioned but wasn't sure which ones. I worked on 8 so far, I can updated my PRs and move ChangeSecret, Delete and Restore ZK to the Admin command. Any others you think? 3. I am also going to search around the code now and take a look now to see if there's anymore commands to fix but if you know of any others let me know. -- 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]
