[GitHub] [accumulo] ctubbsii commented on issue #1776: Check for consistency of shell commands that operate on a table.

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

ctubbsii commented on issue #1776:
URL: https://github.com/apache/accumulo/issues/1776#issuecomment-731479435


   The main concern with adding the option when it's already a required argument is confusion. A user might not know which to specify. They might specify both. What if the user specified both and they were different?! We'd have to add unnecessary complexity to report errors in those cases that we can avoid by limiting it to just the existing required arguments in those cases. I also prefer consistency, but another way to adhere to consistency is to be consistent between versions, rather than be consistent between commands. For those commands, since the required argument is already present, and it would add confusion and complexity to handle, I'm inclined to prefer consistency with previous versions. The scripting use case already works for these, so there's little compelling reason to at the option to these, in my opinion. But, if they are added, it can be done in a separate PR.


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