[HBASE-18605] Clarify cryptic statement in Master#move

• Type: Sub-task
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Component/s: Region Assignment
• Assignee: Unassigned
• Reporter: Michael Stack
• Created: 15/Aug/17 20:26
• Updated: 02/Mar/18 02:31

Description

I don't understand the objective of the below (In review of parent issue, another claimed bafflement so not just me):

 // TODO: What is this? I don't get it.
      if (dest.equals(serverName) && balancer instanceof BaseLoadBalancer
          && !((BaseLoadBalancer)balancer).shouldBeOnMaster(hri)) {
        // To avoid unnecessary region moving later by balancer. Don't put user
        // regions on master.
...

This issue is about cleaning up understanding around what above is for.