Uploaded image for project: 'Jackrabbit Content Repository'
  1. Jackrabbit Content Repository
  2. JCR-3087

InternalVersionHistoryImpl might throw RuntimeException

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Not A Problem
    • None
    • None
    • jackrabbit-core
    • None

    Description

      InternalVersionHistoryImpl.getVersion currently wraps RepositoryExceptions that occur on "reload()" into a RuntimeException. This may cause the RepositoryChecker to abort when attempting to fix a repository, defeating the point of running it.

      I see two options:

      1) Add "throws RepositoryException" to the signature so no wrapping is needed; this will cause similar changes in various interfaces and implementations; but it appears to be straightforward.

      2) Modify the RepositoryChecker to catch RuntimeExceptions as well.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. JCR-3017 Version history recovery fails in case a version does not have a jcr:frozenNode

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            People

              reschke Julian Reschke
              reschke Julian Reschke
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: