Author: phunt
Date: Wed Sep  8 18:28:56 2010
New Revision: 995178

URL: http://svn.apache.org/viewvc?rev=995178&view=rev
Log:
ZOOKEEPER-867. ClientTest is failing on hudson - fd cleanup

Modified:
    hadoop/zookeeper/trunk/CHANGES.txt
    
hadoop/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ClientTest.java

Modified: hadoop/zookeeper/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/CHANGES.txt?rev=995178&r1=995177&r2=995178&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/CHANGES.txt (original)
+++ hadoop/zookeeper/trunk/CHANGES.txt Wed Sep  8 18:28:56 2010
@@ -86,6 +86,8 @@ BUGFIXES: 
   ZOOKEEPER-861. Missing the test SSL certificate used for running junit tests.
   (erwin tam via mahadev)
 
+  ZOOKEEPER-867. ClientTest is failing on hudson - fd cleanup (phunt)
+
 
 IMPROVEMENTS:
   ZOOKEEPER-724. Improve junit test integration - log harness information 

Modified: 
hadoop/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ClientTest.java
URL: 
http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ClientTest.java?rev=995178&r1=995177&r2=995178&view=diff
==============================================================================
--- 
hadoop/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ClientTest.java 
(original)
+++ 
hadoop/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ClientTest.java 
Wed Sep  8 18:28:56 2010
@@ -726,7 +726,10 @@ public class ClientTest extends ClientBa
 
         // if this Assert.fails it means we are not cleaning up after the 
closed
         // sessions.
-        Assert.assertTrue("open fds after test are not significantly higher 
than before",
-                unixos.getOpenFileDescriptorCount() <= initialFdCount + 10);
+        long currentCount = unixos.getOpenFileDescriptorCount();
+        Assert.assertTrue("open fds after test (" + currentCount 
+                + ") are not significantly higher than before ("
+                + initialFdCount + ")",
+                currentCount <= initialFdCount + 10);
     }
 }


Reply via email to