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 <[email protected]>
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 <[email protected]>
    
    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
['"[email protected]" <[email protected]>'].

Reply via email to