[dna-commits] DNA SVN: r338 - trunk/dna-repository/src/main/java/org/jboss/dna/repository/services.
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;