[jboss-cvs] JBossAS SVN: r60691 - branches/Branch_4_2/ejb3/src/main/org/jboss/ejb3/entity.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2007-02-19 23:57:44 -0500 (Mon, 19 Feb 2007)
New Revision: 60691

Modified:
   branches/Branch_4_2/ejb3/src/main/org/jboss/ejb3/entity/JBCCache.java
Log:
In destroy() do remove instead of evict

Modified: branches/Branch_4_2/ejb3/src/main/org/jboss/ejb3/entity/JBCCache.java
===================================================================
--- branches/Branch_4_2/ejb3/src/main/org/jboss/ejb3/entity/JBCCache.java	2007-02-20 04:56:52 UTC (rev 60690)
+++ branches/Branch_4_2/ejb3/src/main/org/jboss/ejb3/entity/JBCCache.java	2007-02-20 04:57:44 UTC (rev 60691)
@@ -210,12 +210,11 @@
 
 	public void destroy() throws CacheException {
 		try {
-			// NOTE : evict() operates locally only (i.e., does not propogate
-			// to any other nodes in the potential cluster).  This is
-			// exactly what is needed when we destroy() here; destroy() is used
-			// as part of the process of shutting down a SessionFactory; thus
-			// these removals should not be propogated
-			cache.evict( regionFqn );
+			// NOTE : Hibernate's class uses evict() but that isn't recursive!
+			//cache.evict( regionFqn );
+            Option opt = new Option();
+            opt.setCacheModeLocal(true);
+            cache.remove(regionFqn, opt);
             
             if (cache.getUseRegionBasedMarshalling() && !isSharedClassLoaderRegion(regionName))
             {