Ted Yu commented on HBASE-18025:

For isRegionInServerManagerStates(), you can directly return the condition 
since the return value is the same as the value of the condition.

For the new test, it should be medium test since minicluster is started.

> CatalogJanitor should collect outdated RegionStates from the AM
> ---------------------------------------------------------------
>                 Key: HBASE-18025
>                 URL: https://issues.apache.org/jira/browse/HBASE-18025
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Esteban Gutierrez
>            Assignee: Esteban Gutierrez
>         Attachments: HBASE-18025.001.patch, HBASE-18025.002.patch, 
> HBASE-18025.003.patch, HBASE-18025.004.patch
> I don't think this will matter on the long run for HBase 2, but at least in 
> branch-1 and the current master we keep in multiple places copies of the 
> region states in the master and this copies include information like the HRI. 
> A problem that we have observed is when region replicas are being used and 
> there is a split, the region replica from parent doesn't get collected from 
> the region states and when the balancer tries to assign the old parent region 
> replica, this will cause the RegionServer to create a new HRI with the 
> details of the parent causing an inconstancy (see HBASE-18024).

This message was sent by Atlassian JIRA

Reply via email to