[bookkeeper] branch master updated: ISSUE #308: LocalBookKeeper.java rethrow Exception in case of failed boot

[zh...@apache.org]

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/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new d5f8247  ISSUE #308: LocalBookKeeper.java rethrow Exception in case of failed boot
d5f8247 is described below

commit d5f824755028e08e07e7f3b326b097148c9b3bec
Author: Enrico Olivelli <eo...@apache.org>
AuthorDate: Thu Jul 27 23:07:13 2017 +0800

    ISSUE #308: LocalBookKeeper.java rethrow Exception in case of failed boot
    
    Do not swallow generic Exceptions during the boot of LocalBookKeeper and let the caller handle errors
    
    Author: Enrico Olivelli <eo...@apache.org>
    
    Reviewers: Jia Zhai <None>, Sijie Guo <None>
    
    This closes #309 from eolivelli/issue-308-localboo-exce, closes #308
---
 .../src/main/java/org/apache/bookkeeper/util/LocalBookKeeper.java        | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/LocalBookKeeper.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/LocalBookKeeper.java
index 903969a..b5f8d72 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/LocalBookKeeper.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/LocalBookKeeper.java
@@ -300,6 +300,7 @@ public class LocalBookKeeper {
         } catch (Exception e) {
             LOG.error("Failed to run {} bookies : zk ensemble = '{}:{}'",
                 new Object[] { numBookies, zkHost, zkPort, e });
+            throw e;
         } finally {
             if (stopOnExit) {
                 cleanupDirectories(bkTmpDirs);

-- 
To stop receiving notification emails like this one, please contact
['"commits@bookkeeper.apache.org" <co...@bookkeeper.apache.org>'].