hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-15631) Backport Regionserver Groups (HBASE-6721) to branch-1
Date Thu, 19 Oct 2017 17:55:00 GMT

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

Andrew Purtell commented on HBASE-15631:

bq. I see that randomAssignment is overridden in RSGroupBasedLoadBalancer, so it might be
relevant, ... hard to say without studying the complete patch.

Thanks. It will be good to get a second pair of eyes on it. 

For what it is worth, I would be ok with leaving that hunk out of the backport for now. Pass
criteria is all unit tests passing. We are going to mark this feature as experimental. The
change may need to be applied later, should we find in cluster chaos testing a problem that
is resolved by it, but it is reasonable to say any change to core for RSgroups should have
a test that covers whatever it is required for. There isn't one for this hunk. We can put
this on a list of post backport follow ups. 

Alternatively, I don't mind including it either, if someone feels strongly about it. That
change to HMaster didn't cause any HBase test failures nor any trouble when RSGroups were
(lightly) tested on a cluster with chaos for an earlier backport to an internal branch based
on 1.3. 

> Backport Regionserver Groups (HBASE-6721) to branch-1 
> ------------------------------------------------------
>                 Key: HBASE-15631
>                 URL: https://issues.apache.org/jira/browse/HBASE-15631
>             Project: HBase
>          Issue Type: New Feature
>    Affects Versions: 1.4.0
>            Reporter: Francis Liu
>            Assignee: Andrew Purtell
>             Fix For: 1.4.0, 1.5.0
>         Attachments: HBASE-15631-branch-1.patch, HBASE-15631-branch-1.patch, HBASE-15631.branch-1.patch,
> Based on dev list discussion backporting region server group should not be an issue as
it does not: 1. destabilize the code. 2. cause backward incompatibility. 

This message was sent by Atlassian JIRA

View raw message