bsglz commented on a change in pull request #2241:
URL: https://github.com/apache/hbase/pull/2241#discussion_r469205948
##########
File path: hbase-shell/src/main/ruby/shell/commands/unassign.rb
##########
@@ -22,8 +22,10 @@ module Commands
class Unassign < Command
def help
<<-EOF
-Unassign a region. Unassign will close region in current location and then
-reopen it again. Pass 'true' to force the unassignment ('force' will clear
+Unassign a region. It could be executed only when region in expected
state(OPEN).
+In addition, you can use "unassigns" which supported by hbck2 to skip the
state check.
+See
https://github.com/apache/hbase-operator-tools/blob/master/hbase-hbck2/README.md
for more info.
Review comment:
Make sense.
BTW, this part seems not correct too, since we do nothing for the "force"
param in server side now.
```
Pass 'true' to force the unassignment ('force' will clear all in-memory
state in
master before the reassign. If results in double assignment use hbck -fix to
resolve.
To be used by experts).
```
----------------------------------------------------------------
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:
us...@infra.apache.org
