You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2011/06/08 01:37:30 UTC
svn commit: r1133197 -
/tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java
Author: markt
Date: Tue Jun 7 23:37:29 2011
New Revision: 1133197
URL: http://svn.apache.org/viewvc?rev=1133197&view=rev
Log:
Enable for async requests.
Depending on how async is used, may see multiple replication requests for a single async request.
Modified:
tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java
Modified: tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java?rev=1133197&r1=1133196&r2=1133197&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ha/tcp/ReplicationValve.java Tue Jun 7 23:37:29 2011
@@ -121,7 +121,7 @@ public class ReplicationValve
// ------------------------------------------------------------- Properties
public ReplicationValve() {
- super(false);
+ super(true);
}
/**
@@ -556,11 +556,13 @@ public class ReplicationValve
}
/**
- * protocol cluster replications stats
+ * Protocol cluster replications stats
* @param requestTime
* @param clusterTime
*/
protected void updateStats(long requestTime, long clusterTime) {
+ // TODO: Async requests may trigger multiple replication requests. How,
+ // if at all, should the stats handle this?
synchronized(this) {
lastSendTime=System.currentTimeMillis();
totalSendTime+=lastSendTime - clusterTime;
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org