tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Remy Maucherat <r...@apache.org>
Subject Re: svn commit: r453771 - in /tomcat/tc6.0.x/trunk: conf/ java/org/apache/catalina/ java/org/apache/catalina/ha/ java/org/apache/catalina/ha/deploy/ java/org/apache/catalina/ha/session/ java/org/apache/catalina/ha/tcp/ java/org/apache/catalina/mbeans/ java...
Date Sun, 08 Oct 2006 15:59:41 GMT
fhanik@apache.org wrote:
> Author: fhanik
> Date: Fri Oct  6 14:17:50 2006
> New Revision: 453771
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=453771
> Log:
> Fixes to the clustering code, some changes in StandardSession broke func, this has been
fixed.
> consolidated the use of a nested <Manager> element
> 

> Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/DeltaSession.java
> URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/DeltaSession.java?view=diff&rev=453771&r1=453770&r2=453771
> ==============================================================================
> --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/DeltaSession.java (original)
> +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/DeltaSession.java Fri Oct
 6 14:17:50 2006
> @@ -43,6 +43,7 @@
>  import org.apache.catalina.util.StringManager;
>  import org.apache.catalina.session.StandardManager;
>  import org.apache.catalina.session.ManagerBase;
> +import java.util.concurrent.atomic.AtomicInteger;
>  
>  /**
>   *
> @@ -103,6 +104,7 @@
>  
>      public DeltaSession(Manager manager) {
>          super(manager);
> +        accessCount = new AtomicInteger();
>          this.resetDeltaRequest();
>      }
>  
> @@ -545,6 +547,7 @@
>          isValid = ( (Boolean) stream.readObject()).booleanValue();
>          thisAccessedTime = ( (Long) stream.readObject()).longValue();
>          version = ( (Long) stream.readObject()).longValue();
> +        this.accessCount = new AtomicInteger();
>          boolean hasPrincipal = stream.readBoolean();
>          principal = null;
>          if (hasPrincipal) {

I was wondering why this was affected by the problem, and it's 
definitely my fault. It would be possible to use the ACTIVITY_CHECK flag 
if you'd want to make usage of accessCount optional. However, I found a 
possible problem: the activate method of the superclass doesn't seem to 
get called, and sessionDidActivate is apparently not processed anywhere 
else either (of course, same for passivate). Is that normal ?

Rémy

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message