Do not serialize the Logger instance

James Moger

2013-12-30 e92cef5abcbf6989e8ce58093b34b3ed909da4b8

 src/main/java/com/gitblit/manager/NotificationManager.java

@@ -32,7 +32,7 @@

import com.gitblit.IStoredSettings;
import com.gitblit.Keys;
import com.gitblit.MailExecutor;
import com.gitblit.service.MailService;

/**
 * The notification manager dispatches notifications.  Currently, email is the
@@ -50,26 +50,27 @@

   private final IStoredSettings settings;

   private final MailExecutor mailExecutor;
   private final MailService mailExecutor;

   public NotificationManager(IStoredSettings settings) {
      this.settings = settings;
      this.mailExecutor = new MailExecutor(settings);
      this.mailExecutor = new MailService(settings);
   }

   @Override
   public IManager setup() {
   public NotificationManager start() {
      if (mailExecutor.isReady()) {
         logger.info("Mail executor is scheduled to process the message queue every 2 minutes.");
         scheduledExecutor.scheduleAtFixedRate(mailExecutor, 1, 2, TimeUnit.MINUTES);
         int period = 2;
         logger.info("Mail service will process the queue every {} minutes.", period);
         scheduledExecutor.scheduleAtFixedRate(mailExecutor, 1, period, TimeUnit.MINUTES);
      } else {
         logger.warn("Mail server is not properly configured.  Mail services disabled.");
         logger.warn("Mail service disabled.");
      }
      return this;
   }

   @Override
   public IManager stop() {
   public NotificationManager stop() {
      scheduledExecutor.shutdownNow();
      return this;
   }