Author: markt
Date: Wed Dec  3 12:05:33 2014
New Revision: 1643100

URL: http://svn.apache.org/r1643100
Log:
Correct reason for adding extra time to expected completion time. This issue is 
not noticeable since TimeoutServlet.ASYNC_TIMEOUT and 
AsyncStartRunnable.THREAD_SLEEP_TIME were both 3000ms.

Modified:
    tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java

Modified: tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java?rev=1643100&r1=1643099&r2=1643100&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java Wed 
Dec  3 12:05:33 2014
@@ -541,8 +541,8 @@ public class TestAsyncContextImpl extend
             long timeoutDelay = TimeoutServlet.ASYNC_TIMEOUT;
             if (asyncDispatch != null && asyncDispatch.booleanValue() &&
                     !completeOnTimeout.booleanValue()) {
-                // Extra timeout in this case
-                timeoutDelay += TimeoutServlet.ASYNC_TIMEOUT;
+                // The async dispatch includes a sleep
+                timeoutDelay += AsyncStartRunnable.THREAD_SLEEP_TIME;
             }
             alvGlobal.validateAccessLog(1, 200, timeoutDelay,
                     timeoutDelay + TIMEOUT_MARGIN + REQUEST_TIME);



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

Reply via email to