[
https://issues.apache.org/jira/browse/HBASE-16377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15414265#comment-15414265
]
Ted Yu commented on HBASE-16377:
--------------------------------
Thanks for the review, Josh.
In order to narrow the scope of change (there is no sign that other part of the
script works incorrectly), I think leaving the {{.to_s}} on currentServer is
safer.
> ServerName check is ineffective in region_mover.rb
> --------------------------------------------------
>
> Key: HBASE-16377
> URL: https://issues.apache.org/jira/browse/HBASE-16377
> Project: HBase
> Issue Type: Bug
> Affects Versions: 1.1.2
> Reporter: Ted Yu
> Assignee: Ted Yu
> Attachments: 16377.branch-1.v1.txt
>
>
> The following was observed during test of region_mover.rb :
> {code}
> 2016-08-08
> 11:17:05,341|beaver.machine|INFO|352|139926954637120|MainThread|2016-08-08
> 11:17:05,340 INFO [RubyThread-9: hbase-client/bin/thread-pool.rb:28]
> region_mover: Moving region hbase:meta,,1.1588230740 (1 of 14) from
> xyz.openstacklocal,16020,1470654716593, to
> server=xyz.openstacklocal,16020,1470654716593
> {code}
> There is check that target server should not be the same as current server:
> {code}
> if currentServer and currentServer == servername
> $LOG.info("Region " + r.getRegionNameAsString() + " (" + counter.to_s +
> " of " + regions.length.to_s + ") already on target server=" +
> servername)
> counter = counter + 1
> next
> end
> {code}
> However, the check is not effective.
> See comparison between object1 and object3:
> http://www.skorks.com/2009/09/ruby-equality-and-object-comparison/
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
