Replace LOGGER by LOG 72/57972/3
authorDavid Suarez <david.suarez.fuentes@ericsson.com>
Mon, 29 May 2017 10:03:08 +0000 (12:03 +0200)
committerRobert Varga <nite@hq.sk>
Mon, 5 Jun 2017 06:12:08 +0000 (06:12 +0000)
Replace LOGGER by LOG to follow the
OpenDaylight recommendations [1].

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

Change-Id: I024bcd5f23a5bdcc177440b175578694c6c471a4
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
(cherry picked from commit 202de5f104dc8149accab4ebf04c3ec8857edff2)

opendaylight/config/yang-jmx-generator-plugin/src/main/java/org/opendaylight/controller/config/yangjmxgenerator/plugin/gofactory/AbsModuleGeneratedObjectFactory.java
opendaylight/md-sal/messagebus-impl/src/main/java/org/opendaylight/controller/config/yang/messagebus/app/impl/MessageBusAppImplModule.java
opendaylight/md-sal/messagebus-util/src/main/java/org/opendaylight/controller/messagebus/app/util/Providers.java

index 7118405deaff7559ec7c60777a8da1c8f0055aa3..0deae078f3a61871aefa640e6ff7bc28075b5a10 100644 (file)
@@ -367,7 +367,7 @@ public class AbsModuleGeneratedObjectFactory {
     }
 
     private static String getLoggerDefinition(FullyQualifiedName fqn) {
-        return format("private static final %s LOGGER = %s.getLogger(%s.class);",
+        return format("private static final %s LOG = %s.getLogger(%s.class);",
                 Logger.class.getCanonicalName(), LoggerFactory.class.getCanonicalName(), fqn);
     }
 
@@ -401,6 +401,6 @@ public class AbsModuleGeneratedObjectFactory {
     }
 
     public String getGetLogger() {
-        return new MethodDefinition(Logger.class.getCanonicalName(), "getLogger", Collections.<Field>emptyList(), "return LOGGER;").toString();
+        return new MethodDefinition(Logger.class.getCanonicalName(), "getLogger", Collections.<Field>emptyList(), "return LOG;").toString();
     }
 }
index a4e551421131498795dff2225e21876f18233619..9f9d737a64180a29b60e22062826afc84c0d68dd 100644 (file)
@@ -19,7 +19,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public class MessageBusAppImplModule extends org.opendaylight.controller.config.yang.messagebus.app.impl.AbstractMessageBusAppImplModule {
-    private static final Logger LOGGER = LoggerFactory.getLogger(MessageBusAppImplModule.class);
+    private static final Logger LOG = LoggerFactory.getLogger(MessageBusAppImplModule.class);
 
     private BundleContext bundleContext;
 
@@ -50,7 +50,7 @@ public class MessageBusAppImplModule extends org.opendaylight.controller.config.
         final DataBroker dataBroker = bindingCtx.getSALService(DataBroker.class);
         final RpcProviderRegistry rpcRegistry = bindingCtx.getSALService(RpcProviderRegistry.class);
         final EventSourceTopology eventSourceTopology = new EventSourceTopology(dataBroker, rpcRegistry);
-        LOGGER.info("Messagebus initialized");
+        LOG.info("Messagebus initialized");
         return eventSourceTopology;
     }
 
index 84c5455ad2631dd3eb0c047bd09c78af3df90292..ca6993664f33abc895b8b2e689c11bc3152b6f80 100644 (file)
@@ -16,14 +16,14 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public class Providers {
-    private static final Logger LOGGER = LoggerFactory.getLogger(Providers.class);
+    private static final Logger LOG = LoggerFactory.getLogger(Providers.class);
 
     public static class BindingAware implements BindingAwareProvider, AutoCloseable {
 
 
         @Override
         public void onSessionInitiated(final BindingAwareBroker.ProviderContext session) {
-            LOGGER.info("BindingAwareBroker.ProviderContext initialized");
+            LOG.info("BindingAwareBroker.ProviderContext initialized");
         }
 
         @Override
@@ -34,7 +34,7 @@ public class Providers {
 
         @Override
         public void onSessionInitiated(final Broker.ProviderSession session) {
-            LOGGER.info("Broker.ProviderSession initialized");
+            LOG.info("Broker.ProviderSession initialized");
         }
 
         @Override