[jbosscache-commits] JBoss Cache SVN: r5838 - core/trunk/src/main/java/org/jboss/cache.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2008-05-14 06:24:03 -0400 (Wed, 14 May 2008)
New Revision: 5838

Modified:
   core/trunk/src/main/java/org/jboss/cache/DataContainer.java
Log:
Updated

Modified: core/trunk/src/main/java/org/jboss/cache/DataContainer.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/DataContainer.java	2008-05-14 09:59:57 UTC (rev 5837)
+++ core/trunk/src/main/java/org/jboss/cache/DataContainer.java	2008-05-14 10:24:03 UTC (rev 5838)
@@ -52,11 +52,17 @@
    {
       this.configuration = configuration;
       this.nodeFactory = nodeFactory;
+
+      // We need to create a root node even at this stage since certain components rely on this being available before
+      // start() is called.
+      // TODO: Investigate which components rely on this being available before start(), and why!
+      createRootNode();
    }
 
    @Start(priority = 12)
    public void createRootNode()
    {
+      if (trace) log.trace("Starting data container");
       // create a new root temporarily.
       NodeSPI tempRoot = nodeFactory.createRootDataNode();
       // if we don't already have a root or the new (temp) root is of a different class (optimistic vs pessimistic) to
@@ -65,7 +71,11 @@
       Class currentRootType = root == null ? null : ((NodeInvocationDelegate) root).getDelegationTarget().getClass();
       Class tempRootType = ((NodeInvocationDelegate) tempRoot).getDelegationTarget().getClass();
 
-      if (!tempRootType.equals(currentRootType)) setRoot(tempRoot);
+      if (!tempRootType.equals(currentRootType))
+      {
+         if (trace) log.trace("Setting root node to an instance of " + tempRootType);
+         setRoot(tempRoot);
+      }
    }
 
    @Stop(priority = 100)