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

DatabaseFileSystem and DatabasePersistenceManager don't allow choice of db schema

    Details

    • Type: Improvement
    • Status: Patch Available
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: jackrabbit-core
    • Labels:
      None
    • Environment:
      All

      Description

      I have a need to store my repository objects under a different db schema than the default for the rdbms (I'm using postgresql, so in my case the default is 'public')

      The current implementation of the DatabasePersistenceManager and DatabaseFileSystem do not support changing the schema.

      Problems:

      • schemaObjectPrefix allows the user to add a table prefix, but you cannot use this to set a schema ie <schema>.table, as the . is stripped out and replaced with an escaped version
      • schema param currently refers to a ddl resource, not what people would naturally think is the param to set the schema for the repository

      Fix:

      • rename the current schema -> schemaDDL
      • add an optional schema param which allows the user to select which schema they want to use
      • improve error messages so that when an incorrect schemaDDL is chosen the user doesn't have to dig through nabble etc to find an answer

        Attachments

        1. JCR-1180_2.2.7.patch
          73 kB
          Stephen Byrne
        2. TablePrefix.patch
          73 kB
          Stephen Byrne
        3. TablePrefix.patch
          73 kB
          Stephen Byrne
        4. postgresql.ddl
          2 kB
          Antonio Mota
        5. postgresql.ddl
          1 kB
          Antonio Mota
        6. DatabaseFileSystem.java
          45 kB
          Antonio Mota
        7. DatabasePersistenceManager.java
          39 kB
          Antonio Mota
        8. jackrabbit-core.patch
          25 kB
          Kev Jackson

          Activity

            People

            • Assignee:
              tmueller Thomas Mueller
              Reporter:
              foamdino Kev Jackson
            • Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated: