Manno15 opened a new pull request #1906:
URL: https://github.com/apache/accumulo/pull/1906
In conjunction with the conversation from #1903, I added a more explicit
error message or granting and revoking System permissions in case the user
forgets to add the '-s' option. Before, It would throw an error with
"Unrecognized permission" and not really tell you why it was unrecognized.
This is how it looked before:
<pre>
org.apache.accumulo.core.util.BadArgumentException: Unrecognized permission
near index 7
revoke System.ALTER_USER -u jeff
</pre>
This is how it looks now:
<pre>
org.apache.accumulo.core.util.BadArgumentException: Missing required option
for granting System Permission: s near index 7
revoke System.ALTER_USER -u jeff
^
</pre>
This is similar formatting to the error message if the user forgets to add
`-u <user>` to a command. I can alter it if there is a preferred
formatting/style choice.
----------------------------------------------------------------
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:
[email protected]
