[jboss-cvs] JBossCache/src/org/jboss/cache/interceptors ...
Manik Surtani
manik at jboss.org
Mon Jul 16 22:03:26 EDT 2007
User: msurtani
Date: 07/07/16 22:03:26
Modified: src/org/jboss/cache/interceptors
CacheLoaderInterceptor.java
ActivationInterceptor.java
Log:
refactored tests
Revision Changes Path
1.91 +2 -2 JBossCache/src/org/jboss/cache/interceptors/CacheLoaderInterceptor.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: CacheLoaderInterceptor.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/interceptors/CacheLoaderInterceptor.java,v
retrieving revision 1.90
retrieving revision 1.91
diff -u -b -r1.90 -r1.91
--- CacheLoaderInterceptor.java 16 Jul 2007 23:43:37 -0000 1.90
+++ CacheLoaderInterceptor.java 17 Jul 2007 02:03:26 -0000 1.91
@@ -25,7 +25,7 @@
* Loads nodes that don't exist at the time of the call into memory from the CacheLoader
*
* @author Bela Ban
- * @version $Id: CacheLoaderInterceptor.java,v 1.90 2007/07/16 23:43:37 msurtani Exp $
+ * @version $Id: CacheLoaderInterceptor.java,v 1.91 2007/07/17 02:03:26 msurtani Exp $
*/
public class CacheLoaderInterceptor extends Interceptor implements CacheLoaderInterceptorMBean
{
@@ -117,9 +117,9 @@
acquireLock = true;
break;
case MethodDeclarations.getNodeMethodLocal_id:
+ case MethodDeclarations.getChildrenNamesMethodLocal_id:
bypassLoadingData = true;
case MethodDeclarations.getKeysMethodLocal_id:
- case MethodDeclarations.getChildrenNamesMethodLocal_id:
case MethodDeclarations.releaseAllLocksMethodLocal_id:
case MethodDeclarations.printMethodLocal_id:
fqn = (Fqn) args[0];
1.62 +2 -4 JBossCache/src/org/jboss/cache/interceptors/ActivationInterceptor.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: ActivationInterceptor.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/interceptors/ActivationInterceptor.java,v
retrieving revision 1.61
retrieving revision 1.62
diff -u -b -r1.61 -r1.62
--- ActivationInterceptor.java 13 Jul 2007 14:53:02 -0000 1.61
+++ ActivationInterceptor.java 17 Jul 2007 02:03:26 -0000 1.62
@@ -26,7 +26,7 @@
* their attributes have been initialized and their children have been loaded in memory.
*
* @author <a href="mailto:{hmesha at novell.com}">{Hany Mesha}</a>
- * @version $Id: ActivationInterceptor.java,v 1.61 2007/07/13 14:53:02 msurtani Exp $
+ * @version $Id: ActivationInterceptor.java,v 1.62 2007/07/17 02:03:26 msurtani Exp $
*/
public class ActivationInterceptor extends CacheLoaderInterceptor implements ActivationInterceptorMBean
{
@@ -107,8 +107,6 @@
}
}
- // if we're here then it's not transactional
-
// CacheLoaderInterceptor normally doesn't load the node
// since CacheStoreInterceptor.put() returns the old value
switch (m.getMethodId())
@@ -152,7 +150,7 @@
log.trace("This is a remove data operation; removing the data from the loader, no activation processing needed.");
loader.removeData(fqn);
}
- else if (((n = getNode(fqn)) != null) && n.isDataLoaded())
+ else if (((n = getNode(fqn)) != null) && n.isDataLoaded() && loader.exists(fqn))
{
// Remove the node from the cache loader if it exists in memory,
// its attributes have been initialized, its children have been loaded,
More information about the jboss-cvs-commits
mailing list