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

Jimmy Xiang commented on HBASE-11530:
-------------------------------------

At first, you can check the history of RegionStates.java. Patch HBASE-10350 
changed the logic a little bit. My understanding is that it's a copy-n-paste 
error. Originally, we add this region to the serverHoldings of lastHost.

The reason we use lastHost instead of serverName, is that those regions not 
assigned have serverName which may be different from lastHost. Here serverName 
is the server where the region is transitioning, for example, opening; while 
lastHost is the server the the region is assigned to previously. The purpose 
here is to make sure the in-mem state is consistent with meta.

> RegionStates adds regions to wrong servers
> ------------------------------------------
>
>                 Key: HBASE-11530
>                 URL: https://issues.apache.org/jira/browse/HBASE-11530
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Jimmy Xiang
>            Assignee: Jimmy Xiang
>             Fix For: 0.99.0
>
>         Attachments: hbase-11530.patch
>
>
> In RegionStates.java, we have (line 332)
> {quote}
> addToServerHoldings(serverName, hri);
> {quote}
> It should be
> {quote}
> addToServerHoldings(lastHost, hri);
> {quote}



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to