[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13485934#comment-13485934
 ] 

Sijie Guo commented on BOOKKEEPER-446:
--------------------------------------

yes, we need to use the original rc. 1) which is the right exception, 2) make 
sync interface and async interface consistent to use same exception.
                
> BookKeeper.createLedger(..) should not mask the error with ZKException
> ----------------------------------------------------------------------
>
>                 Key: BOOKKEEPER-446
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-446
>             Project: Bookkeeper
>          Issue Type: Bug
>          Components: bookkeeper-server
>    Affects Versions: 4.2.0
>            Reporter: Vinay
>
> in {{BookKeeper.createLedger()}} following code is masking the error with 
> ZKException. Should throw the original exception to client.
> {code}        if (counter.getLh() == null) {
>             LOG.error("ZooKeeper error: " + counter.getrc());
>             throw BKException.create(Code.ZKException);
>         }{code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to