jackrabbit-oak-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thomas Mueller (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OAK-189) Swallowed exceptions
Date Mon, 16 Jul 2012 15:00:36 GMT

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

Thomas Mueller commented on OAK-189:
------------------------------------

The reason why I think it's not acceptable is that the exception could be anything, for example
out of disk space, or some internal error. Just silently returning null, without logging,
makes it hard to find the root cause of the problem, because everything else might just look
fine. Some code might accept null as a correct answer, so that the program just behaves somewhat
differently.
                
> Swallowed exceptions
> --------------------
>
>                 Key: OAK-189
>                 URL: https://issues.apache.org/jira/browse/OAK-189
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: jcr
>            Reporter: Thomas Mueller
>
> Exceptions should not be silently swallowed. This is currently done in SessionDelegate$SessionNameMapper,
methods getOakPrefix(), getOakPrefixFromURI(), and getJcrPrefix(). Those methods catch RepositoryException,
don't log by default (only when using debug level), and don't log the exception stack trace
or throw an exception.
> Catching a very wide band of exceptions (RepositoryException) and then simply returning
null is not an acceptable solution in my view.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message