tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Filip Hanik - Dev Lists <devli...@hanik.com>
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 17:42:20 GMT
Remy Maucherat wrote:
> 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 ?
my fault, I was fixing so much stuff, I should call activate when the 
object is done deserializing, I will work more on this during the 
hackathon the next two days

Filip

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


Mime
View raw message