[ 
https://issues.apache.org/jira/browse/HBASE-22658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16883718#comment-16883718
 ] 

Reid Chan commented on HBASE-22658:
-----------------------------------

[~a516072575], I don't see you address sir's comment:
bq. put inside a try/finally... the ruby equivalent...

bq. You need to change scope of class RSGroupAdminClient to public, if its 
scope is private in your code in hbase-rsgroup-xxx.jar
Shouldn't this change include in the pr?

> region_mover.rb  should choose same rsgroup servers as target servers 
> ----------------------------------------------------------------------
>
>                 Key: HBASE-22658
>                 URL: https://issues.apache.org/jira/browse/HBASE-22658
>             Project: HBase
>          Issue Type: Improvement
>          Components: rsgroup, shell
>    Affects Versions: 1.4.10
>            Reporter: liang.feng
>            Priority: Major
>              Labels: gracefulshutdown, region_mover, rsgroup
>         Attachments: HBASE-22658.branch-1.patch
>
>
> There are many retries when i am using graceful_stop.sh to shutdown region 
> server after using regroup, because the target server in a different rsgroup. 
> This makes it slow to graceful shutdown a regionserver. So i think that 
> region_mover.rb  should only choose same rsgroup servers as target servers.
> Region mover is implemented by jruby in hbase1.x and is  implemented by java 
> in hbase2.x . I tried to modify the RegionMover.java class to use the same 
> logic in hbase2.x, but mvn package failed due to hbase-server module and 
> hbase-rsgroup moudle needed to depend on each other, then maven throw a "The 
> projects in the reactor contain a cyclic reference". I couldn't solve it.So I 
> just uploaded patch for hbase1.x .
>  



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to