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

Add an option so skip the "checkSchema" methods

Details

    • New Feature
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • core 1.4.9, 1.5.3
    • 1.6
    • config, jackrabbit-core
    • None

    Description

      Sometimes the "checkSchema" methods in the various components (DB filesystem, DB persistence manager, DB journal) fail to detect that the required tables already exist with as result that the startup fails. (See the mail thread on the dev list: http://jackrabbit.markmail.org/message/jtq2sqis2aceh7ro).
      An option to just skip the checkSchema methods on startup would solve this issue.

      Attachments

        1. JCR-2034-v2.patch
          10 kB
          Martijn Hendriks
        2. JCR-2034.patch
          8 kB
          Martijn Hendriks

        Issue Links

          Activity

            Here's a possible solution that disables the schema checks if you specify the java option org.apache.jackrabbit.core.util.db.schemacheck with value false. By default the schema check is enabled.

            martijnh Martijn Hendriks added a comment - Here's a possible solution that disables the schema checks if you specify the java option org.apache.jackrabbit.core.util.db.schemacheck with value false. By default the schema check is enabled.
            jukkaz Jukka Zitting added a comment -

            Could this be a normal Jackrabbit configuration option instead of a system property?

            jukkaz Jukka Zitting added a comment - Could this be a normal Jackrabbit configuration option instead of a system property?

            Yes, on second thought I think that a normal configuration option is better. I'll create a new patch.

            martijnh Martijn Hendriks added a comment - Yes, on second thought I think that a normal configuration option is better. I'll create a new patch.

            ...and here it is.

            martijnh Martijn Hendriks added a comment - ...and here it is.

            Committed revision 796957.

            martijnh Martijn Hendriks added a comment - Committed revision 796957.
            jukkaz Jukka Zitting added a comment -

            Merged to the 1.x branch in revision 801133.

            jukkaz Jukka Zitting added a comment - Merged to the 1.x branch in revision 801133.

            People

              Unassigned Unassigned
              martijnh Martijn Hendriks
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: