ModeShape
  1. ModeShape
  2. MODE-1511

Change default persistence to store locally on disk

    Details

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

      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.

        Issue Links

          Activity

          Hide
          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
          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
          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
          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.
          Hide
          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
          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.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: