[jira] [Commented] (BOOKKEEPER-908) Case to handle BKLedgerExistException

["Venkateswararao Jujjuri (JV) (JIRA)" <ji...@apache.org>]

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

Venkateswararao Jujjuri (JV) commented on BOOKKEEPER-908:
---------------------------------------------------------

As I explained in the defect,simply a case statement for Code.LedgerExistException is missing in the crearte() function of this class. I noticed it when I decided to take advantage of this error. Patch follows.

> Case to handle BKLedgerExistException
> -------------------------------------
>
>                 Key: BOOKKEEPER-908
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-908
>             Project: Bookkeeper
>          Issue Type: Bug
>          Components: bookkeeper-client
>    Affects Versions: 4.3.2
>            Reporter: Venkateswararao Jujjuri (JV)
>            Assignee: Venkateswararao Jujjuri (JV)
>            Priority: Minor
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> --- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java
> +++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java
> @@ -100,6 +102,8 @@ public abstract class BKException extends Exception {
>              return new BKDuplicateEntryIdException();
>          case Code.TimeoutException:
>              return new BKTimeoutException();
> +        case Code.LedgerExistException:
> +            return new BKLedgerExistException();
>          default:
>              return new BKUnexpectedConditionException();
>          }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)