[GitHub] [accumulo] milleruntime commented on pull request #2914: Deprecate FateCommand

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

milleruntime commented on PR #2914:
URL: https://github.com/apache/accumulo/pull/2914#issuecomment-1248160204

   > I am not sure that we should have two versions of the same command that use separate options in this release. The current command should be deprecated, but could still support the additions made in 2.1.
   > 
   > Having the new options available in the deprecated version seems like it could easy discovery of the new options and serve as a transition. If someone is using the "deprecated" version - they would see the warning, but they may not realize that there are new options available - I guess that adding to the deprecation warning message that new options are available on in the admin version could help - even better would be if the deprecated version help listed the new options as "unavailable" - but it may be easier just to keep the two versions in sync until the deprecated version is removed.
   
   I think you are right that it will be confusing to split the command up between the shell and the admin command. I think it will be better to just deprecate the command here and then move it completely to Admin in the next version.
   


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