hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From te...@apache.org
Subject svn commit: r1231281 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/HConstants.java src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java
Date Fri, 13 Jan 2012 20:07:41 GMT
Author: tedyu
Date: Fri Jan 13 20:07:40 2012
New Revision: 1231281

URL: http://svn.apache.org/viewvc?rev=1231281&view=rev
Log:
HBASE-5143  Fix config typo in pluggable load balancer factory (Harsh J)

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/HConstants.java
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java

Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1231281&r1=1231280&r2=1231281&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Fri Jan 13 20:07:40 2012
@@ -477,6 +477,7 @@ Release 0.92.0 - Unreleased
                it will affect read correctness (Chunhui)
    HBASE-5137  MasterFileSystem.splitLog() should abort even if waitOnSafeMode() throws IOException(Ted)
    HBASE-5121  MajorCompaction may affect scan's correctness (chunhui shen and Lars H)
+   HBASE-5143  Fix config typo in pluggable load balancer factory (Harsh J)
 
   TESTS
    HBASE-4450  test for number of blocks read: to serve as baseline for expected

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/HConstants.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/HConstants.java?rev=1231281&r1=1231280&r2=1231281&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/HConstants.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/HConstants.java Fri Jan 13 20:07:40
2012
@@ -69,6 +69,9 @@ public final class HConstants {
   /** Cluster is in distributed mode or not */
   public static final String CLUSTER_DISTRIBUTED = "hbase.cluster.distributed";
 
+  /** Config for pluggable load balancers */
+  public static final String HBASE_MASTER_LOADBALANCER_CLASS = "hbase.master.loadbalancer.class";
+
   /** Cluster is standalone or pseudo-distributed */
   public static final String CLUSTER_IS_LOCAL = "false";
 

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java?rev=1231281&r1=1231280&r2=1231281&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancerFactory.java Fri
Jan 13 20:07:40 2012
@@ -21,6 +21,7 @@
 package org.apache.hadoop.hbase.master;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.HConstants;
 import org.apache.hadoop.util.ReflectionUtils;
 
 /**
@@ -36,7 +37,9 @@ public class LoadBalancerFactory {
   public static LoadBalancer getLoadBalancer(Configuration conf) {
 
     // Create the balancer
-    Class<? extends LoadBalancer> balancerKlass = conf.getClass("hbase.maser.loadBalancer.class",DefaultLoadBalancer.class,
LoadBalancer.class);
+    Class<? extends LoadBalancer> balancerKlass = conf.getClass(
+        HConstants.HBASE_MASTER_LOADBALANCER_CLASS,
+        DefaultLoadBalancer.class, LoadBalancer.class);
     return ReflectionUtils.newInstance(balancerKlass, conf);
 
   }



Mime
View raw message