Author: markt Date: 2014-12-03 12:05:33 +0000 (Wed, 03 Dec 2014) New Revision: r1643100
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 Index: tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java =================================================================== --- tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java (revision 1643099) +++ tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java (revision 1643100) @@ -541,8 +541,8 @@ 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
