Unnecessary log removal (#2208)
[helix.git] / recipes / rabbitmq-consumer-group / src / main / java / org / apache / helix / recipes / rabbitmq / ConsumerStateModel.java
index 1d693ae8ea6aa9d796c61371a35a62a07fb5c6b1..58f36214197310b9bf7a5f6a2a6735831a334113 100644 (file)
@@ -9,7 +9,7 @@ package org.apache.helix.recipes.rabbitmq;
  * "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
+ *     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
@@ -19,18 +19,19 @@ package org.apache.helix.recipes.rabbitmq;
  * under the License.
  */
 
-import org.apache.log4j.Logger;
-
 import org.apache.helix.NotificationContext;
 import org.apache.helix.model.Message;
 import org.apache.helix.participant.statemachine.StateModel;
 import org.apache.helix.participant.statemachine.StateModelInfo;
 import org.apache.helix.participant.statemachine.Transition;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
-@StateModelInfo(initialState = "OFFLINE", states = { "ONLINE", "ERROR" })
-public class ConsumerStateModel extends StateModel
-{
-  private static Logger LOG = Logger.getLogger(ConsumerStateModel.class);
+@StateModelInfo(initialState = "OFFLINE", states = {
+    "ONLINE", "ERROR"
+})
+public class ConsumerStateModel extends StateModel {
+  private static Logger LOG = LoggerFactory.getLogger(ConsumerStateModel.class);
 
   private final String _consumerId;
   private final String _partition;
@@ -38,20 +39,17 @@ public class ConsumerStateModel extends StateModel
   private final String _mqServer;
   private ConsumerThread _thread = null;
 
-  public ConsumerStateModel(String consumerId, String partition, String mqServer)
-  {
+  public ConsumerStateModel(String consumerId, String partition, String mqServer) {
     _partition = partition;
     _consumerId = consumerId;
     _mqServer = mqServer;
   }
 
   @Transition(to = "ONLINE", from = "OFFLINE")
-  public void onBecomeOnlineFromOffline(Message message, NotificationContext context)
-  {
+  public void onBecomeOnlineFromOffline(Message message, NotificationContext context) {
     LOG.debug(_consumerId + " becomes ONLINE from OFFLINE for " + _partition);
 
-    if (_thread == null)
-    {
+    if (_thread == null) {
       LOG.debug("Starting ConsumerThread for " + _partition + "...");
       _thread = new ConsumerThread(_partition, _mqServer, _consumerId);
       _thread.start();
@@ -62,54 +60,46 @@ public class ConsumerStateModel extends StateModel
 
   @Transition(to = "OFFLINE", from = "ONLINE")
   public void onBecomeOfflineFromOnline(Message message, NotificationContext context)
-      throws InterruptedException
-  {
+      throws InterruptedException {
     LOG.debug(_consumerId + " becomes OFFLINE from ONLINE for " + _partition);
 
-    if (_thread != null)
-    {
+    if (_thread != null) {
       LOG.debug("Stopping " + _consumerId + " for " + _partition + "...");
 
       _thread.interrupt();
       _thread.join(2000);
       _thread = null;
-      LOG.debug("Stopping " +  _consumerId + " for " + _partition + " done");
+      LOG.debug("Stopping " + _consumerId + " for " + _partition + " done");
 
     }
   }
 
   @Transition(to = "DROPPED", from = "OFFLINE")
-  public void onBecomeDroppedFromOffline(Message message, NotificationContext context)
-  {
+  public void onBecomeDroppedFromOffline(Message message, NotificationContext context) {
     LOG.debug(_consumerId + " becomes DROPPED from OFFLINE for " + _partition);
   }
 
   @Transition(to = "OFFLINE", from = "ERROR")
-  public void onBecomeOfflineFromError(Message message, NotificationContext context)
-  {
+  public void onBecomeOfflineFromError(Message message, NotificationContext context) {
     LOG.debug(_consumerId + " becomes OFFLINE from ERROR for " + _partition);
   }
 
   @Override
-  public void reset()
-  {
+  public void reset() {
     LOG.warn("Default reset() invoked");
-    
-    if (_thread != null)
-    {
+
+    if (_thread != null) {
       LOG.debug("Stopping " + _consumerId + " for " + _partition + "...");
 
       _thread.interrupt();
-      try
-      {
+      try {
         _thread.join(2000);
-      } catch (InterruptedException e)
-      {
+      } catch (InterruptedException e) {
         // TODO Auto-generated catch block
         e.printStackTrace();
       }
       _thread = null;
-      LOG.debug("Stopping " +  _consumerId + " for " + _partition + " done");
+      LOG.debug("Stopping " + _consumerId + " for " + _partition + " done");
 
     }
   }