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

Mikhail Antonov commented on HBASE-10915:
-----------------------------------------

A note - ClosedRegionHandler on HM side doesn't really use ZK API, it calls 
assignment manager, the only suspicious piece is :

{code}
if (this.assignmentManager.getZKTable().
        isDisablingOrDisabledTable(this.regionInfo.getTable())) {
      assignmentManager.offlineDisabledRegion(regionInfo);
      return;
    }
{code}
HBASE-7767 proposes to get rid of ZKTable, yet it doesn't seem to be worked on 
now. I think we can either leave this as is (in ClosedRegionHandler), or 
transform ZKTable into interface for like TableStateManager and impl?

> Decouple region closing from ZooKeeper
> --------------------------------------
>
>                 Key: HBASE-10915
>                 URL: https://issues.apache.org/jira/browse/HBASE-10915
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Zookeeper
>            Reporter: Mikhail Antonov
>            Assignee: Mikhail Antonov
>         Attachments: HBASE-10915.patch, HBASE-10915.patch, HBASE-10915.patch, 
> HBASE-10915.patch, HBASE-10915.patch, HBASE-10915.patch, HBASE-10915.patch, 
> HBASE-10915.patch
>
>
> Decouple CloseRegionHandler class and code using it (HRegionServer, 
> ProtobufUtil) from ZK API.



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

Reply via email to