ZOOKEEPER-2822: Wrong ObjectName about MBeanServer in JMX module

apache · Jun 27, 2017 · 8654cc0 · 8654cc0
1 parent a0eba7a
commit 8654cc0
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/java/main/org/apache/zookeeper/jmx/ManagedUtil.java b/src/java/main/org/apache/zookeeper/jmx/ManagedUtil.java
@@ -33,13 +33,13 @@
 public class ManagedUtil {
     private static final Logger LOG = LoggerFactory.getLogger(ManagedUtil.class);
 
-    private static final boolean isLog4jJmxEnabled() {
+    private static boolean isLog4jJmxEnabled() {
         boolean enabled = false;
 
         try {
             Class.forName("org.apache.log4j.spi.LoggerRepository");
 
-            if (Boolean.getBoolean("zookeeper.jmx.log4j.disable") == true) {
+            if (Boolean.getBoolean("zookeeper.jmx.log4j.disable")) {
                 LOG.info("Log4j found but jmx support is disabled.");
             } else {
                 enabled = true;
@@ -71,7 +71,7 @@ public static void registerLog4jMBeans() throws JMException {
                 // org.apache.log4j.jmx.HierarchyDynamicMBean hdm = new org.apache.log4j.jmx.HierarchyDynamicMBean();
                 Object hdm = Class.forName("org.apache.log4j.jmx.HierarchyDynamicMBean").newInstance();
 
-                ObjectName mbo = new ObjectName("log4j:hiearchy=default");
+                ObjectName mbo = new ObjectName("log4j:hierarchy=default");
                 mbs.registerMBean(hdm, mbo);
 
                 // Add the root logger to the Hierarchy MBean