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