Replace logger and log by LOG
authorDavid Suarez <david.suarez.fuentes@ericsson.com>
Tue, 30 May 2017 19:11:21 +0000 (21:11 +0200)
committerTom Pantelis <tompantelis@gmail.com>
Thu, 1 Jun 2017 12:52:33 +0000 (12:52 +0000)
Replace logger and log by LOG to follow the
OpenDaylight recommendations [1].

[1]
https://wiki.opendaylight.org/view/BestPractices/Logging_Best_Practices

Change-Id: I63787ccee5950bebbc8c3769885574593a666809
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
src/test/java/org/opendaylight/protocol/framework/Session.java
src/test/java/org/opendaylight/protocol/framework/SimpleDispatcher.java
src/test/java/org/opendaylight/protocol/framework/SimpleSessionListener.java

index 22ad93065655369e08d4bec3b755abf5e268f219..b1defd25cb98e3152ce033f92a43cf5f204bc629 100644 (file)
@@ -16,7 +16,7 @@ import com.google.common.collect.Lists;
 
 public class Session extends AbstractProtocolSession<SimpleMessage> {
 
-    private static final Logger logger = LoggerFactory.getLogger(Session.class);
+    private static final Logger LOG = LoggerFactory.getLogger(Session.class);
 
     public final List<SimpleMessage> msgs = Lists.newArrayList();
 
@@ -29,19 +29,19 @@ public class Session extends AbstractProtocolSession<SimpleMessage> {
 
     @Override
     public void handleMessage(final SimpleMessage msg) {
-        logger.debug("Message received: {}", msg.getMessage());
+        LOG.debug("Message received: {}", msg.getMessage());
         this.up = true;
         this.msgs.add(msg);
-        logger.debug(this.msgs.size() + "");
+        LOG.debug(this.msgs.size() + "");
     }
 
     @Override
     public void endOfInput() {
-        logger.debug("End of input reported.");
+        LOG.debug("End of input reported.");
     }
 
     @Override
     protected void sessionUp() {
-        logger.debug("Session up reported.");
+        LOG.debug("Session up reported.");
     }
 }
index d83738520cccc1e7bc24b4f97cb31c30ff6e51ad..314fd42c7d37c2865fdb5842afdcfa6c3d62ce87 100644 (file)
@@ -22,7 +22,7 @@ import org.slf4j.LoggerFactory;
 import com.google.common.base.Preconditions;
 
 public class SimpleDispatcher extends AbstractDispatcher<SimpleSession, SimpleSessionListener> {
-    private static final Logger logger = LoggerFactory.getLogger(SimpleDispatcher.class);
+    private static final Logger LOG = LoggerFactory.getLogger(SimpleDispatcher.class);
 
     private final SessionNegotiatorFactory<SimpleMessage, SimpleSession, SimpleSessionListener> negotiatorFactory;
     private final ChannelOutboundHandler encoder = new SimpleMessageToByteEncoder();
@@ -39,7 +39,7 @@ public class SimpleDispatcher extends AbstractDispatcher<SimpleSession, SimpleSe
             channel.pipeline().addLast(new SimpleByteToMessageDecoder());
             channel.pipeline().addLast("negotiator", negotiatorFactory.getSessionNegotiator(listenerFactory, channel, promise));
             channel.pipeline().addLast(encoder);
-            logger.debug("initialization completed for channel {}", channel);
+            LOG.debug("initialization completed for channel {}", channel);
         }
 
     }
index 5d167566f2635fc8e839e4d5f633ecc05ed04704..8db14f7f5a41288fb48aec885cd6fe7fac338d95 100644 (file)
@@ -17,7 +17,7 @@ import org.slf4j.LoggerFactory;
  * Simple Session Listener that is notified about messages and changes in the session.
  */
 public class SimpleSessionListener implements SessionListener<SimpleMessage, SimpleSession, TerminationReason> {
-    private static final Logger logger = LoggerFactory.getLogger(SimpleSessionListener.class);
+    private static final Logger LOG = LoggerFactory.getLogger(SimpleSessionListener.class);
 
     public List<SimpleMessage> messages = new ArrayList<>();
 
@@ -27,7 +27,7 @@ public class SimpleSessionListener implements SessionListener<SimpleMessage, Sim
 
     @Override
     public void onMessage(final SimpleSession session, final SimpleMessage message) {
-        logger.debug("Received message: " + message.getClass() + " " + message);
+        LOG.debug("Received message: " + message.getClass() + " " + message);
         this.messages.add(message);
     }