[dna-commits] DNA SVN: r338 - trunk/dna-repository/src/main/java/org/jboss/dna/repository/services.

[dna-commits at lists.jboss.org]

Author: rhauch
Date: 2008-07-03 11:40:44 -0400 (Thu, 03 Jul 2008)
New Revision: 338

Modified:
   trunk/dna-repository/src/main/java/org/jboss/dna/repository/services/AbstractServiceAdministrator.java
Log:
DNA-115 - Create federation service 
http://jira.jboss.com/jira/browse/DNA-115

Added trace logging to the service administrator.

Modified: trunk/dna-repository/src/main/java/org/jboss/dna/repository/services/AbstractServiceAdministrator.java
===================================================================
--- trunk/dna-repository/src/main/java/org/jboss/dna/repository/services/AbstractServiceAdministrator.java	2008-07-03 11:16:46 UTC (rev 337)
+++ trunk/dna-repository/src/main/java/org/jboss/dna/repository/services/AbstractServiceAdministrator.java	2008-07-03 15:40:44 UTC (rev 338)
@@ -25,6 +25,7 @@
 import net.jcip.annotations.GuardedBy;
 import net.jcip.annotations.ThreadSafe;
 import org.jboss.dna.common.i18n.I18n;
+import org.jboss.dna.common.util.Logger;
 import org.jboss.dna.repository.RepositoryI18n;
 
 /**
@@ -38,6 +39,7 @@
 
     private volatile State state;
     private final I18n serviceName;
+    private final Logger logger;
 
     protected AbstractServiceAdministrator( I18n serviceName,
                                             State initialState ) {
@@ -45,6 +47,7 @@
         assert serviceName != null;
         this.state = initialState;
         this.serviceName = serviceName;
+        this.logger = Logger.getLogger(getClass());
     }
 
     /**
@@ -118,8 +121,10 @@
             case STARTED:
                 break;
             case PAUSED:
+                logger.trace("Starting \"{0}\"", getServiceName());
                 doStart(this.state);
                 this.state = State.STARTED;
+                logger.trace("Started \"{0}\"", getServiceName());
                 break;
             case SHUTDOWN:
             case TERMINATED:
@@ -153,8 +158,10 @@
     public synchronized ServiceAdministrator pause() {
         switch (this.state) {
             case STARTED:
+                logger.trace("Pausing \"{0}\"", getServiceName());
                 doPause(this.state);
                 this.state = State.PAUSED;
+                logger.trace("Paused \"{0}\"", getServiceName());
                 break;
             case PAUSED:
                 break;
@@ -167,8 +174,8 @@
 
     /**
      * Implementation of the functionality to switch to the paused state. This method is only called if the state from which the
-     * service is transitioning is appropriate ({@link ServiceAdministrator.State#STARTED}). This method does nothing by
-     * default, and should be overridden if needed.
+     * service is transitioning is appropriate ({@link ServiceAdministrator.State#STARTED}). This method does nothing by default,
+     * and should be overridden if needed.
      * 
      * @param fromState the state from which this service is transitioning; never null
      * @throws IllegalStateException if the service is such that it cannot be transitioned from the supplied state
@@ -190,8 +197,10 @@
         switch (this.state) {
             case STARTED:
             case PAUSED:
+                logger.trace("Initiating shutdown of \"{0}\"", getServiceName());
                 this.state = State.SHUTDOWN;
                 doShutdown(this.state);
+                logger.trace("Initiated shutdown of \"{0}\"", getServiceName());
                 isTerminated();
                 break;
             case SHUTDOWN:
@@ -262,6 +271,7 @@
             case SHUTDOWN:
                 if (doCheckIsTerminated()) {
                     this.state = State.TERMINATED;
+                    logger.trace("Service \"{0}\" has terminated", getServiceName());
                     return true;
                 }
                 return false;