Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 3673F1688F4 for ; Wed, 9 Aug 2017 07:51:22 +0200 (CEST) Received: (qmail 6990 invoked by uid 500); 9 Aug 2017 05:51:20 -0000 Mailing-List: contact commits-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list commits@hbase.apache.org Received: (qmail 5401 invoked by uid 99); 9 Aug 2017 05:51:19 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 09 Aug 2017 05:51:19 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 4830AF69B3; Wed, 9 Aug 2017 05:51:17 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: busbey@apache.org To: commits@hbase.apache.org Date: Wed, 09 Aug 2017 05:51:38 -0000 Message-Id: In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [22/27] hbase git commit: HBASE-18511 Default no regions on master HBASE-18511 Default no regions on master Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/a8e0267c Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/a8e0267c Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/a8e0267c Branch: refs/heads/HBASE-18467 Commit: a8e0267c00409da7e3e03cfb34168787873ef075 Parents: 4376233 Author: Michael Stack Authored: Sat Aug 5 15:28:40 2017 +0800 Committer: Michael Stack Committed: Tue Aug 8 12:11:02 2017 +0800 ---------------------------------------------------------------------- .../hbase/master/balancer/BaseLoadBalancer.java | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/a8e0267c/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java index 8f5b6f5..445f80f 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java @@ -1111,12 +1111,8 @@ public abstract class BaseLoadBalancer implements LoadBalancer { private static final Random RANDOM = new Random(System.currentTimeMillis()); private static final Log LOG = LogFactory.getLog(BaseLoadBalancer.class); - // Regions of these tables are put on the master by default. - private static final String[] DEFAULT_TABLES_ON_MASTER = - new String[] {AccessControlLists.ACL_TABLE_NAME.getNameAsString(), - TableName.NAMESPACE_TABLE_NAME.getNameAsString(), - TableName.META_TABLE_NAME.getNameAsString()}; - + // Master carries no regions by default. + private static final String[] DEFAULT_TABLES_ON_MASTER = new String[] {}; public static final String TABLES_ON_MASTER = "hbase.balancer.tablesOnMaster"; @@ -1127,18 +1123,18 @@ public abstract class BaseLoadBalancer implements LoadBalancer { protected MasterServices services; /** - * By default, regions of some small system tables such as meta, - * namespace, and acl are assigned to the active master. If you don't - * want to assign any region to the active master, you need to - * configure "hbase.balancer.tablesOnMaster" to "none". + * By default, master carries no regions. This method returns null. + * If you want master to carry system tables say, then set + * TABLES_ON_MASTER to AccessControlLists.ACL_TABLE_NAME.getNameAsString(), + * TableName.NAMESPACE_TABLE_NAME.getNameAsString(), + * TableName.META_TABLE_NAME.getNameAsString() */ protected static String[] getTablesOnMaster(Configuration conf) { String valueString = conf.get(TABLES_ON_MASTER); if (valueString == null) { return DEFAULT_TABLES_ON_MASTER; } - valueString = valueString.trim(); - if (valueString.equalsIgnoreCase("none")) { + if (valueString == null || valueString.trim().isEmpty()) { return null; } return StringUtils.getStrings(valueString);