Author: markt
Date: Sun Apr 28 22:04:06 2013
New Revision: 1476846

URL: http://svn.apache.org/r1476846
Log:
Port unit test changes from trunk that should allow ASF CI system to complete 
without error

Modified:
    tomcat/tc7.0.x/trunk/   (props changed)
    tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/LocalStrings.properties
    tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/StandardWrapper.java
    tomcat/tc7.0.x/trunk/test/org/apache/catalina/loader/TestVirtualContext.java

Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
  Merged /tomcat/trunk:r1476761,1476805,1476815

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/LocalStrings.properties?rev=1476846&r1=1476845&r2=1476846&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/LocalStrings.properties 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/LocalStrings.properties 
Sun Apr 28 22:04:06 2013
@@ -261,7 +261,7 @@ standardWrapper.statusTitle=Tomcat Error
 standardWrapper.unavailable=Marking servlet {0} as unavailable
 standardWrapper.unloadException=Servlet {0} threw unload() exception
 standardWrapper.unloading=Cannot allocate servlet {0} because it is being 
unloaded
-standardWrapper.waiting=Waiting for {0} instance(s) to be deallocated
+standardWrapper.waiting=Waiting for {0} instance(s) to be deallocated for 
Servlet [{1}]
 threadLocalLeakPreventionListener.lifecycleEvent.error=Exception processing 
lifecycle event {0}
 threadLocalLeakPreventionListener.containerEvent.error=Exception processing 
container event {0}
 

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/StandardWrapper.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/StandardWrapper.java?rev=1476846&r1=1476845&r2=1476846&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/StandardWrapper.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/StandardWrapper.java Sun 
Apr 28 22:04:06 2013
@@ -1448,7 +1448,8 @@ public class StandardWrapper extends Con
             while ((nRetries < 21) && (countAllocated.get() > 0)) {
                 if ((nRetries % 10) == 0) {
                     log.info(sm.getString("standardWrapper.waiting",
-                                          countAllocated.toString()));
+                                          countAllocated.toString(),
+                                          getName()));
                 }
                 try {
                     Thread.sleep(delay);

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/loader/TestVirtualContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/loader/TestVirtualContext.java?rev=1476846&r1=1476845&r2=1476846&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/loader/TestVirtualContext.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/loader/TestVirtualContext.java 
Sun Apr 28 22:04:06 2013
@@ -297,7 +297,13 @@ public class TestVirtualContext extends 
     private void assertPageContains(String pageUrl, String expectedBody,
         int expectedStatus) throws IOException {
         ByteChunk res = new ByteChunk();
-        int sc = getUrl("http://localhost:"; + getPort() + pageUrl, res, 3000,
+        // Note: With a read timeout of 3s the ASF CI buildbot was consistently
+        //       seeing failures with this test. The failures were due to the
+        //       JSP initialisation taking longer than the read timeout. The
+        //       root cause of this is the frequent poor IO performance of the
+        //       VM running the buildbot instance. Increasing this to 10s 
should
+        //       avoid these failures.
+        int sc = getUrl("http://localhost:"; + getPort() + pageUrl, res, 10000,
                 null, null);
 
         assertEquals(expectedStatus, sc);



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

Reply via email to