[
https://issues.apache.org/jira/browse/HBASE-22658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16888271#comment-16888271
]
Andrew Purtell commented on HBASE-22658:
----------------------------------------
It looks like this patch is getting dinged for all the rubocop notions of bad
syntax in the entire file. Those issues can be sorted out on another issue.
The changes lgtm but I am concerned about lack of test coverage. How do we know
they actually work? Ruby is so dynamic, the code may look plausible, but we
have no idea what kind of runtime exceptions may result.
Can we add a test here? Or at least post some evidence, through manual testing,
that the change works as intended.
> 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.002.patch,
> 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)
