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

Ted Yu commented on HBASE-4213:
-------------------------------

{code}
+        boolean currentBalanceSwitch = master.isBalancerEnabaled();
+        if (this.master.isBalancerEnabaled()) {
+          LOG.info("Load Balancer enabled. Disabling load balancer for " +
+              "instant schema change");
+          this.master.balanceSwitch(false);
+        }
{code}
I think there is no need to introduce a new method. master.balanceSwitch() 
returns the old value which can be stored in currentBalanceSwitch.
{code}
+        while(true) {
+          if (!masterSchemaChangeTracker.isSchemaChangeNodeExists(
+              Bytes.toString(tableName))) {
+            break;
+          }
+        }
{code}
Should we introduce fault tolerance logic in the above loop ? E.g. if a region 
server goes down and thus would not acknowledge completion of the schema change.

> Support instant schema updates with out master's intervention (i.e with out 
> enable/disable and bulk assign/unassign)
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-4213
>                 URL: https://issues.apache.org/jira/browse/HBASE-4213
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Subbu M Iyer
>            Assignee: Subbu M Iyer
>             Fix For: 0.92.0
>
>         Attachments: HBASE-4213-Instant_schema_change.patch, 
> HBASE-4213_Instant_schema_change_-Version_2_.patch, 
> HBASE_Instant_schema_change-version_3_.patch
>
>
> This Jira is a slight variation in approach to what is being done as part of 
> https://issues.apache.org/jira/browse/HBASE-1730
> Support instant schema updates such as Modify Table, Add Column, Modify 
> Column operations:
> 1. With out enable/disabling the table.
> 2. With out bulk unassign/assign of regions.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to