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

Jean-Marc Spaggiari commented on HBASE-10127:
---------------------------------------------

Sound good to me.

some extra spaces after
{code}
List<RegionPlan> plans = new ArrayList<RegionPlan>();
       <=== Here
{code}

And here:
{code}
     return getMaster().balance();
   }
+  <=== Here
{code}

For the java code, can we say "Balance table's regions" or "Balance regions of 
a table"?

getMaster() is deprecated. Any chance to replace it? (Not 100% required for the 
scope of this patch)

Can you do the trunk version to trigger Hadoop QA?

> support balance table
> ---------------------
>
>                 Key: HBASE-10127
>                 URL: https://issues.apache.org/jira/browse/HBASE-10127
>             Project: HBase
>          Issue Type: Improvement
>          Components: master, shell
>    Affects Versions: 0.94.14
>            Reporter: cuijianwei
>         Attachments: HBASE-10127-0.94-v1.patch
>
>
> HMaster provides a rpc interface : 'balance()' to balance all the regions 
> among region servers in the cluster. Sometimes, we might want to balance all 
> the regions belonging to a table while keeping the region assignments of 
> other tables. This demand may reveal in a shared cluster where we want to 
> balance regions for one application's table without affecting other 
> applications. Therefore, will it be useful if we extend the current 
> 'balance()' interface to only balance regions of the same table? 



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to