You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by zh...@apache.org on 2018/09/25 08:43:07 UTC

[pulsar] branch master updated: [tests] ignore exceptions during shutting down test cases (#2645)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6658891  [tests] ignore exceptions during shutting down test cases (#2645)
6658891 is described below

commit 6658891eaa49189a431cf4aede0e35453b737ca7
Author: Sijie Guo <gu...@gmail.com>
AuthorDate: Tue Sep 25 01:43:02 2018 -0700

    [tests] ignore exceptions during shutting down test cases (#2645)
---
 .../java/org/apache/pulsar/broker/service/BrokerBkEnsemblesTests.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerBkEnsemblesTests.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerBkEnsemblesTests.java
index 94b2226..7a06823 100644
--- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerBkEnsemblesTests.java
+++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerBkEnsemblesTests.java
@@ -125,8 +125,7 @@ public class BrokerBkEnsemblesTests {
             pulsar.close();
             bkEnsemble.stop();
         } catch (Throwable t) {
-            LOG.error("Error cleaning up broker test setup state", t);
-            Assert.fail("Broker test cleanup failed");
+            LOG.warn("Error cleaning up broker test setup state", t);
         }
     }