[GitHub] [accumulo] Manno15 commented on issue #1903: Remove '-s' option when granting System permission

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

Manno15 commented on issue #1903:
URL: https://github.com/apache/accumulo/issues/1903#issuecomment-772871032


   > @Manno15 issues should generally not be questions, but clear well-defined tasks whenever possible. This is especially true for those with the "good first issue" label, because those can attract new contributors. If a new contributor tries to work on a task with an open ended question, it can create a very confusing and unpleasant experience for them if the community hasn't yet decided on a course of action. If you want to get feedback or consensus from the community on an idea, the best place to do that is on the developer mailing list, rather than in the body of a new issue.
   
   My apologies. that makes sense. I will keep that in mind in the future so I don't make the same mistake. 
   
   > A better option would probably be to recognize the missing -s and provide a more helpful error message. I think originally, the mutual exclusivity was enforced, the error message should have been something like "missing one of -s, -t" or similar. If that was relaxed at any point, it may have introduced the worse error message.
   
   I will look into this tomorrow. I am open to closing this issue as is. 
   
   


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

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