Jackrabbit Oak
  1. Jackrabbit Oak
  2. OAK-48

MicroKernel.getNodes() should return null for not existing nodes instead of throwing an exception

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.2.1
    • Component/s: mk
    • Labels:
      None

      Description

      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

      1. OAK-48.patch
        6 kB
        Michael Dürig

        Issue Links

          Activity

          Alex Parvulescu made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Alex Parvulescu made changes -
          Fix Version/s 0.2.1 [ 12321252 ]
          Fix Version/s 0.2 [ 12320344 ]
          Thomas Mueller made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Thomas Mueller made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Assignee Stefan Guggisberg [ stefan@jira ] Thomas Mueller [ tmueller ]
          Stefan Guggisberg made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 0.2 [ 12320344 ]
          Resolution Fixed [ 1 ]
          Stefan Guggisberg made changes -
          Assignee Stefan Guggisberg [ stefan@jira ]
          Michael Dürig made changes -
          Attachment OAK-48.patch [ 12522231 ]
          Michael Dürig made changes -
          Field Original Value New Value
          Link This issue is part of OAK-11 [ OAK-11 ]
          Michael Dürig created issue -

            People

            • Assignee:
              Thomas Mueller
              Reporter:
              Michael Dürig
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development