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

stack commented on HBASE-4308:
------------------------------

Is this check the wrong way round Ram?

{code}
+    if (!openedNodeDeleted) {
+      if (this.assignmentManager.getZKTable().isDisablingOrDisabledTable(
+          regionInfo.getTableNameAsString())) {
+        debugLog(regionInfo, "Opened region "
+            + regionInfo.getRegionNameAsString() + " but "
+            + "this table is disabled, triggering close of region");
+        assignmentManager.unassign(regionInfo);
+      }
     }
{code}

If we failed to delete the znode, only then you check if disabled?  Won't 
openedNodeDeleted be true if all goes well and this is when you want to check 
if region is of a disabling table?

It looks like in old code that we checked table disabling whether we succeeded 
znode delete or not?

Otherwise, I'm +1 on this patch (You can do fixup if I'm right and go ahead and 
commit)


                
> Race between RegionOpenedHandler and AssignmentManager
> ------------------------------------------------------
>
>                 Key: HBASE-4308
>                 URL: https://issues.apache.org/jira/browse/HBASE-4308
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.92.0
>            Reporter: Todd Lipcon
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 0.92.0
>
>         Attachments: HBASE-4308.patch, HBASE-4308_1.patch, HBASE-4308_2.patch
>
>
> When the master is processing a ZK event for REGION_OPENED, it calls delete() 
> on the znode before it removes the node from RegionsInTransition. If the 
> notification of that delete comes back into AssignmentManager before the 
> region is removed from RIT, you see an error like:
> 2011-08-30 17:43:29,537 WARN  [main-EventThread] 
> master.AssignmentManager(861): Node deleted but still in RIT: 
> .META.,,1.1028785192 state=OPEN, ts=1314751409532, 
> server=todd-w510,55655,1314751396840
> Not certain if it causes issues, but it's a concerning log message.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to