Changed logger to slf4j

Changed all instances of Sys.out or PolicyLogger to
slf4j.

Issue-ID: POLICY-178
Change-Id: Ia91af1deb18a9b5c466f6f8a6c546c17c54e14ad
Signed-off-by: Magnusen, Drew (dm741q) <dm741q@att.com>

1 files changed, 21 insertions, 9 deletions

diff --git a/integrity-monitor/src/main/java/org/onap/policy/common/im/jmx/ComponentAdmin.java b/integrity-monitor/src/main/java/org/onap/policy/common/im/jmx/ComponentAdmin.java
index 8ffb1b74..c9b1c1df 100644
--- a/integrity-monitor/src/main/java/org/onap/policy/common/im/jmx/ComponentAdmin.java
+++ b/integrity-monitor/src/main/java/org/onap/policy/common/im/jmx/ComponentAdmin.java
@@ -32,7 +32,8 @@ import javax.management.MalformedObjectNameException;
 import javax.management.NotCompliantMBeanException;
 import javax.management.ObjectName;
 
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import org.onap.policy.common.im.IntegrityMonitor;
 import org.onap.policy.common.im.StateManagement;
@@ -41,7 +42,7 @@ import org.onap.policy.common.im.StateManagement;
  * Base class for component MBeans.
  */
 public class ComponentAdmin implements ComponentAdminMBean {
-	private static final Logger logger = Logger.getLogger(ComponentAdmin.class.getName());
+	private static final Logger logger = LoggerFactory.getLogger(ComponentAdmin.class.getName());
 
 	private final String name;
 	private MBeanServer registeredMBeanServer;
@@ -69,7 +70,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 		try {
 			register();
 		} catch (Exception e) {
-			logger.info("Failed to register ComponentAdmin MBean");
+			if(logger.isDebugEnabled()){
+				logger.debug("Failed to register ComponentAdmin MBean");
+			}
 			throw e;
 		}
 	}
@@ -87,7 +90,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 			InstanceAlreadyExistsException, NotCompliantMBeanException {
 
 
-			logger.info("Registering " + name + " MBean");
+		if(logger.isDebugEnabled()){
+			logger.debug("Registering {} MBean", name);
+		}
 
 
 		MBeanServer mbeanServer = findMBeanServer();
@@ -99,8 +104,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 		ObjectName objectName = new ObjectName(name);
 
 		if (mbeanServer.isRegistered(objectName)) {
-			logger.info("Unregistering a previously registered "
-				+ name + " MBean");
+			if(logger.isDebugEnabled()){
+				logger.debug("Unregistering a previously registered {} MBean", name);
+			}
 			mbeanServer.unregisterMBean(objectName);
 		}
 
@@ -186,7 +192,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 	@Override
 	public void test() throws Exception {
 		// Call evaluateSanity on IntegrityMonitor to run the test
-		logger.info("test() called...");
+		if(logger.isDebugEnabled()){
+			logger.debug("test() called...");
+		}
 		if (integrityMonitor != null) {
 			integrityMonitor.evaluateSanity();
 		}
@@ -199,7 +207,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 
 	@Override
 	public void lock() throws Exception {
-		logger.info("lock() called...");
+		if(logger.isDebugEnabled()){
+			logger.debug("lock() called...");
+		}
 		if (stateManager != null) {
 			stateManager.lock();
 		}
@@ -211,7 +221,9 @@ public class ComponentAdmin implements ComponentAdminMBean {
 
 	@Override
 	public void unlock() throws Exception {
-		logger.info("unlock() called...");
+		if(logger.isDebugEnabled()){
+			logger.debug("unlock() called...");
+		}
 		if (stateManager != null) {
 			stateManager.unlock();
 		}