[HBASE-5016] LoadBalancerFactory - "maser" in package name of dynamically loaded balancer class?

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Assignee: Unassigned
• Reporter: Doug Meil
• Created: 13/Dec/11 17:47
• Updated: 12/Jun/22 19:53
• Resolved: 30/Aug/12 16:19

Description

Shouldn't this... "hbase.maser.loadBalancer.class" be something like "hbase.master.loadBalancer.class"? What's with "maser"?

/**
 * The class that creates a load balancer from a conf.
 */
public class LoadBalancerFactory {

  /**
   * Create a loadblanacer from the given conf.
   * @param conf
   * @return A {@link LoadBalancer}
   */
  public static LoadBalancer getLoadBalancer(Configuration conf) {

    // Create the balancer
    Class<? extends LoadBalancer> balancerKlass = conf.getClass("hbase.maser.loadBalancer.class",DefaultLoadBalancer.class, LoadBalancer.class);
    return ReflectionUtils.newInstance(balancerKlass, conf);

  }
}