Github user breed commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/561#discussion_r201303257
--- Diff: src/java/main/org/apache/zookeeper/jmx/MBeanRegistry.java ---
@@ -124,9 +124,7 @@ private void unregister(String path,ZKMBeanInfo bean)
throws JMException {
return;
if (!bean.isHidden()) {
final ObjectName objName = makeObjectName(path, bean);
- if (LOG.isInfoEnabled()) {
- LOG.info("Unregister MBean [{}]", objName);
- }
+ LOG.debug("Unregister MBean [{}]", objName);
--- End diff --
any idea why this was in an if clause before. i don't think it was needed,
but just checking.
---
