Uploaded image for project: 'ModeShape'
  1. ModeShape
  2. MODE-1511

Change default persistence to store locally on disk

    Details

    • Type: Feature Request
    • Status: Closed (View Workflow)
    • Priority: Blocker
    • Resolution: Rejected
    • Affects Version/s: 3.0.0.Alpha5
    • Fix Version/s: None
    • Component/s: JCR
    • Labels:
      None

      Description

      Change the default behavior for a configuration so that all data is stored locally on the file system within the directory where the application is being run. So for example, we could create a directory with the repository name, and in that directory create a "store" directory (where we'd persist the Infinispan cache), a "binaries" directory for binary storage, and an "indexes" directory for the Lucene indexes. The benefit is that all the repository data survives a restart with no extra configuration, and the memory footprint is smaller. It also would (hopefully) be clear to the user that if these directories were created and they don't like the location, they can simply change them by altering their repository configuration.

        Gliffy Diagrams

          Issue Links

            Activity

            rhauch Randall Hauch created issue -
            Hide
            hchiorean Horia Chiorean added a comment -

            If we change the default behavior, we need to make sure that our tests still run using the in-memory settings. We already have a TestingEnviroment and some base classes we're using in our tests, so we could define a default (test) configuration there.

            We just need to be sure then that the tests use the default config.

            Show
            hchiorean Horia Chiorean added a comment - If we change the default behavior, we need to make sure that our tests still run using the in-memory settings. We already have a TestingEnviroment and some base classes we're using in our tests, so we could define a default (test) configuration there. We just need to be sure then that the tests use the default config.
            Hide
            rhauch Randall Hauch added a comment -

            We also have an AbstractJcrRepositoryTest (with SingleUseAbstractTest and MultiUseAbstractTest subclasses) that we're hopefully using in as many places as we can. Perhaps we need to use them more consistently, or perhaps we need to update how we set up our repositories for unit testing.

            Show
            rhauch Randall Hauch added a comment - We also have an AbstractJcrRepositoryTest (with SingleUseAbstractTest and MultiUseAbstractTest subclasses) that we're hopefully using in as many places as we can. Perhaps we need to use them more consistently, or perhaps we need to update how we set up our repositories for unit testing.
            rhauch Randall Hauch made changes -
            Field Original Value New Value
            Assignee Randall Hauch [ rhauch ]
            rhauch Randall Hauch made changes -
            Fix Version/s 3.0.0.Beta2 [ 12318691 ]
            Fix Version/s 3.0.0.Beta1 [ 12319652 ]
            Forum Reference https://community.jboss.org/message/740832 https://community.jboss.org/message/740832
            rhauch Randall Hauch made changes -
            Fix Version/s 3.0.0.Beta3 [ 12319783 ]
            Fix Version/s 3.0.0.Beta2 [ 12318691 ]
            Forum Reference https://community.jboss.org/message/740832 https://community.jboss.org/message/740832
            Hide
            rhauch Randall Hauch added a comment -

            After significant thought, we no longer think this is ideal: a configuration file that doesn't specified where information should be persisted should not persist information. Instead, we'll create better examples and documentation for file-based persistence.

            Show
            rhauch Randall Hauch added a comment - After significant thought, we no longer think this is ideal: a configuration file that doesn't specified where information should be persisted should not persist information. Instead, we'll create better examples and documentation for file-based persistence.
            rhauch Randall Hauch made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Fix Version/s 3.0.0.Beta3 [ 12319783 ]
            Resolution Rejected [ 2 ]
            rhauch Randall Hauch made changes -
            Link This issue is related to MODE-1589 [ MODE-1589 ]
            rhauch Randall Hauch made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

              People

              • Assignee:
                Unassigned
                Reporter:
                rhauch Randall Hauch
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development