This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 0c9f152  Cleanup client correctly in 
TestCompatOldClients#testFencingOldClient
0c9f152 is described below

commit 0c9f152336e8af012589b713242e46f475f7055d
Author: Ivan Kelly <iv...@apache.org>
AuthorDate: Mon Feb 19 21:55:16 2018 -0800

    Cleanup client correctly in TestCompatOldClients#testFencingOldClient
    
    Previously we had been mistakenly cleaning up the classloader twice.
    
    Author: Ivan Kelly <iv...@apache.org>
    
    Reviewers: Enrico Olivelli <eolive...@gmail.com>, Sijie Guo 
<si...@apache.org>
    
    This closes #1176 from ivankelly/kill-client-bc-test
---
 .../apache/bookkeeper/tests/backwardcompat/TestCompatOldClients.groovy | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/tests/backward-compat/current-server-old-clients/src/test/groovy/org/apache/bookkeeper/tests/backwardcompat/TestCompatOldClients.groovy
 
b/tests/backward-compat/current-server-old-clients/src/test/groovy/org/apache/bookkeeper/tests/backwardcompat/TestCompatOldClients.groovy
index a527567..5a29fa4 100644
--- 
a/tests/backward-compat/current-server-old-clients/src/test/groovy/org/apache/bookkeeper/tests/backwardcompat/TestCompatOldClients.groovy
+++ 
b/tests/backward-compat/current-server-old-clients/src/test/groovy/org/apache/bookkeeper/tests/backwardcompat/TestCompatOldClients.groovy
@@ -103,7 +103,8 @@ class TestCompatOldClients {
         } finally {
             oldBK.close()
             oldCL.close()
-            fencingCL.close()
+
+            fencingBK.close()
             fencingCL.close()
         }
     }

-- 
To stop receiving notification emails like this one, please contact
si...@apache.org.

Reply via email to