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

Gomathivinayagam Muthuvinayagam commented on HBASE-11699:
---------------------------------------------------------

I will send out the code review shortly. 

Just a few pointers what I have done

1) There will be one config file, which will have entries for including and 
excluding region servers.
    <hostname> = allow/deny
2) Added functions in HBaseAdmin to refresh the config file. Initially when the 
ServerManager class is initiated, the config file will be refreshed. So if 
there are any region servers which are online and also excluded will be expired.

3) Changed HMaster to support the functions, also regenerated protobuf classes. 


> Region servers exclusion list to HMaster.
> -----------------------------------------
>
>                 Key: HBASE-11699
>                 URL: https://issues.apache.org/jira/browse/HBASE-11699
>             Project: HBase
>          Issue Type: New Feature
>          Components: Admin, Client, regionserver, Zookeeper
>    Affects Versions: 0.98.3
>            Reporter: Gomathivinayagam Muthuvinayagam
>            Priority: Minor
>             Fix For: 0.99.0, 2.0.0, 0.98.6
>
>   Original Estimate: 96h
>  Remaining Estimate: 96h
>
> Currently HBase does not support adding set of region servers to be in the 
> exclusion list. So that administrators can prevent accidental startups of 
> some region servers to join the cluster. There was initially some work done, 
> and it is available in https://issues.apache.org/jira/browse/HBASE-3833. It 
> was not done after that. 
> I am planning to contribute it as a patch, and I would like to do some 
> improvements as well. Instead of storing the exclusion entries on a file, I 
> am planning to store it on zookeeper. Can anyone suggest thoughts on this? 



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

Reply via email to