Fixes for sonar critical issues

[policy/engine.git] / PolicyEngineUtils / src / main / java / org / onap / policy / utils / BusPublisher.java

diff --git a/PolicyEngineUtils/src/main/java/org/onap/policy/utils/BusPublisher.java b/PolicyEngineUtils/src/main/java/org/onap/policy/utils/BusPublisher.java
index 792c389..3190aa0 100644 (file)
--- a/PolicyEngineUtils/src/main/java/org/onap/policy/utils/BusPublisher.java
+++ b/PolicyEngineUtils/src/main/java/org/onap/policy/utils/BusPublisher.java
@@ -1,3 +1,23 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * PolicyEngineUtils
+ * ================================================================================
+ * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
 package org.onap.policy.utils;
 
 import java.util.ArrayList;
@@ -6,6 +26,8 @@ import java.util.Properties;
 import java.util.concurrent.TimeUnit;
 
 import org.onap.policy.common.logging.eelf.PolicyLogger;
+import org.onap.policy.common.logging.flexlogger.FlexLogger;
+import org.onap.policy.common.logging.flexlogger.Logger;
 
 import com.att.nsa.mr.client.impl.MRSimplerBatchPublisher;
 import com.att.nsa.mr.test.clients.ProtocolTypeConstants;
@@ -31,6 +53,7 @@ public interface BusPublisher {
         * DmaapClient library wrapper
         */
        public static class DmaapPublisherWrapper implements BusPublisher {
+           private static Logger logger = FlexLogger.getLogger(DmaapPublisherWrapper.class);
                /**
                 * MR based Publisher
                 */             
@@ -74,16 +97,16 @@ public interface BusPublisher {
                 */
                @Override
                public void close() {
-                       if (PolicyLogger.isInfoEnabled())
-                               PolicyLogger.info(DmaapPublisherWrapper.class.getName(), 
+                       if (logger.isInfoEnabled())
+                           logger.info(DmaapPublisherWrapper.class.getName()+ 
                                                  "CREATION: " + this);
                        
                        try {
                                this.publisher.close(1, TimeUnit.SECONDS);
                        } catch (Exception e) {
-                               PolicyLogger.warn(DmaapPublisherWrapper.class.getName(), 
+                           logger.warn(DmaapPublisherWrapper.class.getName()+ 
                                                  "CLOSE: " + this + " because of " + 
-                                                                 e.getMessage());
+                                                                 e.getMessage(), e);
                        }
                }