[jira] [Comment Edited] (HBASE-20160) TestRestartCluster.testRetainAssignmentOnRestart uses the wrong condition to decide whether the assignment is finished

["stack (JIRA)" <ji...@apache.org>]

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

stack edited comment on HBASE-20160 at 3/8/18 7:33 PM:
-------------------------------------------------------

 +1 for for branch-2.0+ Nice cleanup.


was (Author: stack):
+1 for for branch-2.0+ Nice cleanup.

> TestRestartCluster.testRetainAssignmentOnRestart uses the wrong condition to decide whether the assignment is finished
> ----------------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-20160
>                 URL: https://issues.apache.org/jira/browse/HBASE-20160
>             Project: HBase
>          Issue Type: Bug
>          Components: Region Assignment
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Major
>             Fix For: 2.0.0
>
>         Attachments: HBASE-20160.patch
>
>
> {code}
>     // Wait till master is initialized and all regions are assigned
>     RegionStates regionStates = master.getAssignmentManager().getRegionStates();
>     int expectedRegions = regionToRegionServerMap.size() + 1;
>     while (!master.isInitialized()
>         || regionStates.getRegionAssignments().size() != expectedRegions) {
>       Threads.sleep(100);
>     }
> {code}
> Actually this does not mean the assignment is finished. In AMv2, we will load the region state from meta when restarting, so the regionStates.getRegionAssignments will reach the expected count soon. But this is just the old location. After that, we will continue to execute the ServerCrashProcedure to deal with the reassignment. That's why sometimes we may fail with
> {noformat}
> java.lang.AssertionError: Values should be different. Actual: 1520478964169
> 	at org.apache.hadoop.hbase.master.TestRestartCluster.testRetainAssignmentOnRestart(TestRestartCluster.java:215)
> {noformat}
> We just read the old location from meta since the ServerCrashProcedure has not been finished yet, but we want to confirm that the region is on the same host and port but a new RS.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)