[
https://issues.apache.org/jira/browse/HBASE-6721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13492575#comment-13492575
]
Ted Yu commented on HBASE-6721:
-------------------------------
In GroupInfoManager:
{code}
+ boolean moveServers(Set<String> hostPort, String srcGroup, String dstGroup)
throws IOException;
{code}
Please add javadoc for moveServers() method.
For GroupInfoManagerImpl , indentation should be 2 spaces.
For moveServers:
{code}
+ for(String el: hostPort) {
+ foundOne = foundOne || src.removeServer(el);
+ dst.addServer(el);
{code}
If foundOne is true in the middle of the loop, would the call to
src.removeServer(el) be skipped ? That would lead to server being in both
groups.
> RegionServer Group based Assignment
> -----------------------------------
>
> Key: HBASE-6721
> URL: https://issues.apache.org/jira/browse/HBASE-6721
> Project: HBase
> Issue Type: New Feature
> Reporter: Francis Liu
> Assignee: Vandana Ayyalasomayajula
> Fix For: 0.96.0
>
> Attachments: HBASE-6721_94_2.patch, HBASE-6721_94_3.patch,
> HBASE-6721_94.patch, HBASE-6721_94.patch, HBASE-6721-DesigDoc.pdf
>
>
> In multi-tenant deployments of HBase, it is likely that a RegionServer will
> be serving out regions from a number of different tables owned by various
> client applications. Being able to group a subset of running RegionServers
> and assign specific tables to it, provides a client application a level of
> isolation and resource allocation.
> The proposal essentially is to have an AssignmentManager which is aware of
> RegionServer groups and assigns tables to region servers based on groupings.
> Load balancing will occur on a per group basis as well.
> This is essentially a simplification of the approach taken in HBASE-4120. See
> attached document.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
