[Jboss-cvs] JBossAS SVN: r56317 - branches/Branch_4_0/tomcat/src/main/org/jboss/web/tomcat/tc5/session

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

Author: bstansberry at jboss.com
Date: 2006-08-27 23:36:25 -0400 (Sun, 27 Aug 2006)
New Revision: 56317

Modified:
   branches/Branch_4_0/tomcat/src/main/org/jboss/web/tomcat/tc5/session/ClusteredSession.java
Log:
Don't replicate metadata just because isNew changed

Modified: branches/Branch_4_0/tomcat/src/main/org/jboss/web/tomcat/tc5/session/ClusteredSession.java
===================================================================
--- branches/Branch_4_0/tomcat/src/main/org/jboss/web/tomcat/tc5/session/ClusteredSession.java	2006-08-28 02:08:12 UTC (rev 56316)
+++ branches/Branch_4_0/tomcat/src/main/org/jboss/web/tomcat/tc5/session/ClusteredSession.java	2006-08-28 03:36:25 UTC (rev 56317)
@@ -1059,7 +1059,13 @@
    public void setNew(boolean isNew)
    {
       super.setNew(isNew);
-      sessionMetadataDirty();
+      // Don't replicate metadata just 'cause its the second request
+      // The only effect of this is if someone besides a request 
+      // deserializes metadata from the distributed cache, this 
+      // field may be out of date.
+      // If a request accesses the session, the access() call will
+      // set isNew=false, so the request will see the correct value
+      // sessionMetadataDirty();
    }
    
    public void setValid(boolean isValid)