Unnecessary log removal (#2208)
[helix.git] / recipes / rabbitmq-consumer-group / src / main / java / org / apache / helix / recipes / rabbitmq / ConsumerStateModel.java
index 27d935e52ee3ad6a9f328d806025d0f05acf7fa6..58f36214197310b9bf7a5f6a2a6735831a334113 100644 (file)
@@ -1,17 +1,37 @@
 package org.apache.helix.recipes.rabbitmq;
 
-import org.apache.log4j.Logger;
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you 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.
+ */
 
 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;
@@ -19,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();
@@ -43,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");
 
     }
   }