Github user asdf2014 commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/294#discussion_r234997204
--- Diff:
zookeeper-server/src/main/java/org/apache/zookeeper/jmx/ManagedUtil.java ---
@@ -69,9 +69,13 @@ public static void registerLog4jMBeans() throws
JMException {
try {
// Create and Register the top level Log4J MBean
// org.apache.log4j.jmx.HierarchyDynamicMBean hdm = new
org.apache.log4j.jmx.HierarchyDynamicMBean();
- Object hdm =
Class.forName("org.apache.log4j.jmx.HierarchyDynamicMBean").getDeclaredConstructor().newInstance();
+ Object hdm =
Class.forName("org.apache.log4j.jmx.HierarchyDynamicMBean").getConstructor().newInstance();
- ObjectName mbo = new ObjectName("log4j:hiearchy=default");
+ String mbean = System.getenv("zookeeper.jmx.log4j.mbean");
--- End diff --
@eolivelli Okay, I will modify it.
---
