[ 
https://issues.apache.org/jira/browse/JCR-1269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12662295#action_12662295
 ] 

Felix Meschberger commented on JCR-1269:
----------------------------------------

@angela: I see your point, this is why I suggested to have the BaseException 
extend the RepositoryException

@Jukka: The problem is consistent exception handling and understanding what 
this is all about. In that sense its kind of a categorization of the 
exceptions. Of course the exceptions previously extending BaseException could 
just extend RepositoryException. But having them extend Exception is IMHO wrong.


Anyway, since these are just internal exceptions, I don't care too much -- 
unless this would lead to method declarations of the form

    public type someMethod() throws Exception

> Stop using BaseException
> ------------------------
>
>                 Key: JCR-1269
>                 URL: https://issues.apache.org/jira/browse/JCR-1269
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Jukka Zitting
>            Assignee: Jukka Zitting
>            Priority: Minor
>             Fix For: 1.6.0
>
>         Attachments: BaseException.patch
>
>
> The o.a.j.BaseException class is deprecated (since JCR-1169) and not caught 
> anywhere, so there's no need to keep using it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to