jackrabbit-oak-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Guggisberg (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OAK-48) MicroKernel.getNodes() should return null for not existing nodes instead of throwing an exception
Date Fri, 13 Apr 2012 20:43:16 GMT

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

Stefan Guggisberg commented on OAK-48:
--------------------------------------

> Also, it seems there is not test case yet.

yes, there is:

http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-it/mk/src/main/java/org/apache/jackrabbit/mk/test/MicroKernelIT.java?r1=1325827&r2=1325826&view=diff&pathrev=1325827
                
> MicroKernel.getNodes() should return null for not existing nodes instead of throwing
an exception
> -------------------------------------------------------------------------------------------------
>
>                 Key: OAK-48
>                 URL: https://issues.apache.org/jira/browse/OAK-48
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: mk
>            Reporter: Michael Dürig
>            Assignee: Thomas Mueller
>             Fix For: 0.2
>
>         Attachments: OAK-48.patch
>
>
> As discussed on the list [1, 2] exceptions should only be used for exceptional cases.
Requesting a not existing node through Microkernel.getNodes() should therefore rather return
null instead of throwing an exception.
> [1] http://markmail.org/thread/agibgcdjv756m53u
> [2] http://markmail.org/message/gfbmogwr6mrhe2pm

--
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