[ https://issues.apache.org/jira/browse/ZOOKEEPER-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709496#action_12709496 ]
Mahadev konar commented on ZOOKEEPER-358: ----------------------------------------- falvio, adding a test would a good thing so that it prevents breaking the code later. also, specially in this case, since its a api semantics change, also a test for this should nto be mroe than 2 lines of code? :) > Throw exception when ledger does not exist > ------------------------------------------ > > Key: ZOOKEEPER-358 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-358 > Project: Zookeeper > Issue Type: Improvement > Components: contrib-bookkeeper > Affects Versions: 3.1.1 > Reporter: Luca Telloli > Assignee: Flavio Paiva Junqueira > Priority: Minor > Attachments: ZOOKEEPER-358.patch > > > Currently, openLedger() in the BookKeeper client returns null if the ledger > ID does not exist on ZK. Maybe it would be better to throw a specific > exception so it can be handled by the client side. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.