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); } }