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

Enis Soztutar commented on HBASE-9514:
--------------------------------------

A couple of comments:
 - should we rename AM.acquireLock -> AM.acquireRegionLock() 
 - Why not do this for meta? 
{code}
+        if (!region.isMetaRegion() &&
+            regionStates.wasRegionOnDeadServer(encodedName)) {
{code}

Is it safe to expire a server like this. It means the master cannot connect to 
it, but it may still have the zk lease. 
{code}
+            } else {
+              LOG.info(server + " is not reachable, expire it");
+              serverManager.expireServer(server);
+            }
{code}

- Should we rename RegionStates.logSplit() -> markAssignable() or something 
like it. 

Is this timeout intended to be active?
{code}
+  @Test //(timeout=60000)
{code}
                
> Prevent region from assigning before log splitting is done
> ----------------------------------------------------------
>
>                 Key: HBASE-9514
>                 URL: https://issues.apache.org/jira/browse/HBASE-9514
>             Project: HBase
>          Issue Type: Bug
>          Components: Region Assignment
>            Reporter: Jimmy Xiang
>            Assignee: Jimmy Xiang
>            Priority: Blocker
>         Attachments: trunk-9514_v1.patch
>
>
> If a region is assigned before log splitting is done by the server shutdown 
> handler, the edits belonging to this region in the hlogs of the dead server 
> will be lost.
> Generally this is not an issue if users don't assign/unassign a region from 
> hbase shell or via hbase admin. These commands are marked for experts only in 
> the hbase shell help too.  However, chaos monkey doesn't care.
> If we can prevent from assigning such regions in a bad time, it would make 
> things a little safer.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to