clarax edited a comment on pull request #69: URL: https://github.com/apache/hbase-operator-tools/pull/69#issuecomment-1055851862
@wchevreuil @ndimiduk Thank you for the feedback. Sorry I shelved this for other priorities. I am back to finish this. The changes are: 1. added a common -i option so any command that takes a list of argument can take a list of input files. 2. updated the -h message and README. 3. added a unit test for the generic option and test this option for assigns and unassigns. For other commands, since they all share the same code path, I don't see a need to add more ut. The existing ut for other command don't touch generic options either. 4. for commands that take two arguments(table|resgion state), I opened a separate jira and pr for taking the pairs in a list of files. https://issues.apache.org/jira/browse/HBASE-26785 but later decided to include here. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
