[exo-jcr-commits] exo-jcr SVN: r743 - jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/dataflow/persistent.

do-not-reply at jboss.org do-not-reply at jboss.org
Wed Nov 18 05:06:52 EST 2009


Author: tolusha
Date: 2009-11-18 05:06:51 -0500 (Wed, 18 Nov 2009)
New Revision: 743

Modified:
   jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/dataflow/persistent/JBossCacheWorkspaceDataManager.java
Log:
EXOJCR-201: prepare changes log list

Modified: jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/dataflow/persistent/JBossCacheWorkspaceDataManager.java
===================================================================
--- jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/dataflow/persistent/JBossCacheWorkspaceDataManager.java	2009-11-18 09:19:29 UTC (rev 742)
+++ jcr/branches/1.12.0-JBC/component/core/src/main/java/org/exoplatform/services/jcr/impl/dataflow/persistent/JBossCacheWorkspaceDataManager.java	2009-11-18 10:06:51 UTC (rev 743)
@@ -18,9 +18,13 @@
  */
 package org.exoplatform.services.jcr.impl.dataflow.persistent;
 
+import org.exoplatform.services.jcr.dataflow.ChangesLogIterator;
+import org.exoplatform.services.jcr.dataflow.CompositeChangesLog;
 import org.exoplatform.services.jcr.dataflow.ItemState;
 import org.exoplatform.services.jcr.dataflow.ItemStateChangesLog;
+import org.exoplatform.services.jcr.dataflow.PlainChangesLog;
 import org.exoplatform.services.jcr.dataflow.ReadOnlyThroughChanges;
+import org.exoplatform.services.jcr.dataflow.TransactionChangesLog;
 import org.exoplatform.services.jcr.datamodel.QPath;
 import org.exoplatform.services.jcr.impl.Constants;
 import org.exoplatform.services.jcr.impl.dataflow.TransientItemData;
@@ -28,8 +32,10 @@
 import org.exoplatform.services.jcr.storage.WorkspaceDataContainer;
 import org.exoplatform.services.jcr.storage.WorkspaceStorageConnection;
 
+import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.Iterator;
+import java.util.List;
 import java.util.Set;
 
 import javax.jcr.RepositoryException;
@@ -69,85 +75,102 @@
       WorkspaceStorageConnection thisConnection = null;
       WorkspaceStorageConnection systemConnection = null;
 
+      // prepare changes log list
+      List<PlainChangesLog> chengesLogList = new ArrayList<PlainChangesLog>();
+      if (changesLog instanceof PlainChangesLog)
+      {
+         chengesLogList.add((PlainChangesLog)changesLog);
+      }
+      else if (changesLog instanceof CompositeChangesLog)
+      {
+         for (ChangesLogIterator iter = ((CompositeChangesLog)changesLog).getLogIterator(); iter.hasNextLog();)
+         {
+            chengesLogList.add(iter.nextLog());
+         }
+      }
+
       try
       {
-         for (Iterator<ItemState> iter = changesLog.getAllStates().iterator(); iter.hasNext();)
+         for (PlainChangesLog curChangesLog : chengesLogList)
          {
-            ItemState itemState = iter.next();
+            for (Iterator<ItemState> iter = curChangesLog.getAllStates().iterator(); iter.hasNext();)
+            {
+               ItemState itemState = iter.next();
 
-            if (!itemState.isPersisted())
-               continue;
+               if (!itemState.isPersisted())
+                  continue;
 
-            long start = System.currentTimeMillis();
+               long start = System.currentTimeMillis();
 
-            TransientItemData data = (TransientItemData)itemState.getData();
+               TransientItemData data = (TransientItemData)itemState.getData();
 
-            WorkspaceStorageConnection conn = null;
-            if (isSystemDescendant(data.getQPath()))
-            {
-               conn = systemConnection == null
-               // we need system connection but it's not exist
-                  ? systemConnection = (systemDataContainer != dataContainer
-                  // if it's different container instances
-                     ? systemDataContainer.equals(dataContainer) && thisConnection != null
-                     // but container confugrations are same and non-system connnection open
-                        // reuse this connection as system
-                        ? systemDataContainer.reuseConnection(thisConnection)
-                        // or open one new system
-                        : systemDataContainer.openConnection()
-                     // else if it's same container instances (system and this)
-                     : thisConnection == null
-                     // and non-system connection doens't exist - open it
-                        ? thisConnection = dataContainer.openConnection()
-                        // if already open - use it
-                        : thisConnection)
-                  // system connection opened - use it
-                  : systemConnection;
-            }
-            else
-            {
-               conn = thisConnection == null
-               // we need this conatiner conection
-                  ? thisConnection = (systemDataContainer != dataContainer
-                  // if it's different container instances
-                     ? dataContainer.equals(systemDataContainer) && systemConnection != null
-                     // but container confugrations are same and system connnection open
-                        // reuse system connection as this
-                        ? dataContainer.reuseConnection(systemConnection)
-                        // or open one new
-                        : dataContainer.openConnection()
-                     // else if it's same container instances (system and this)
-                     : systemConnection == null
-                     // and system connection doens't exist - open it
-                        ? systemConnection = dataContainer.openConnection()
-                        // if already open - use it
-                        : systemConnection)
-                  // this connection opened - use it
-                  : thisConnection;
-            }
+               WorkspaceStorageConnection conn = null;
+               if (isSystemDescendant(data.getQPath()))
+               {
+                  conn = systemConnection == null
+                  // we need system connection but it's not exist
+                     ? systemConnection = (systemDataContainer != dataContainer
+                     // if it's different container instances
+                        ? systemDataContainer.equals(dataContainer) && thisConnection != null
+                        // but container confugrations are same and non-system connnection open
+                           // reuse this connection as system
+                           ? systemDataContainer.reuseConnection(thisConnection)
+                           // or open one new system
+                           : systemDataContainer.openConnection()
+                        // else if it's same container instances (system and this)
+                        : thisConnection == null
+                        // and non-system connection doens't exist - open it
+                           ? thisConnection = dataContainer.openConnection()
+                           // if already open - use it
+                           : thisConnection)
+                     // system connection opened - use it
+                     : systemConnection;
+               }
+               else
+               {
+                  conn = thisConnection == null
+                  // we need this conatiner conection
+                     ? thisConnection = (systemDataContainer != dataContainer
+                     // if it's different container instances
+                        ? dataContainer.equals(systemDataContainer) && systemConnection != null
+                        // but container confugrations are same and system connnection open
+                           // reuse system connection as this
+                           ? dataContainer.reuseConnection(systemConnection)
+                           // or open one new
+                           : dataContainer.openConnection()
+                        // else if it's same container instances (system and this)
+                        : systemConnection == null
+                        // and system connection doens't exist - open it
+                           ? systemConnection = dataContainer.openConnection()
+                           // if already open - use it
+                           : systemConnection)
+                     // this connection opened - use it
+                     : thisConnection;
+               }
 
-            data.increasePersistedVersion();
+               data.increasePersistedVersion();
 
-            if (itemState.isAdded())
-            {
-               doAdd(data, conn, addedNodes);
+               if (itemState.isAdded())
+               {
+                  doAdd(data, conn, addedNodes);
+               }
+               else if (itemState.isUpdated())
+               {
+                  doUpdate(data, conn);
+               }
+               else if (itemState.isDeleted())
+               {
+                  doDelete(data, conn);
+               }
+               else if (itemState.isRenamed())
+               {
+                  doRename(data, conn, addedNodes);
+               }
+
+               if (LOG.isDebugEnabled())
+                  LOG.debug(ItemState.nameFromValue(itemState.getState()) + " " + (System.currentTimeMillis() - start)
+                     + "ms, " + data.getQPath().getAsString());
             }
-            else if (itemState.isUpdated())
-            {
-               doUpdate(data, conn);
-            }
-            else if (itemState.isDeleted())
-            {
-               doDelete(data, conn);
-            }
-            else if (itemState.isRenamed())
-            {
-               doRename(data, conn, addedNodes);
-            }
-
-            if (LOG.isDebugEnabled())
-               LOG.debug(ItemState.nameFromValue(itemState.getState()) + " " + (System.currentTimeMillis() - start)
-                  + "ms, " + data.getQPath().getAsString());
          }
          if (thisConnection != null)
             thisConnection.commit();



More information about the exo-jcr-commits mailing list