Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-7158

Users shouldn't be able to change the number of retained generations

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.9.0, 1.10.0, 1.8.1
    • segment-tar
    • None

    Description

      Currently, SegmentGCOptions implements a validation check on the number of retained generations that prevents callers to set it to a value smaller than two. Instead, some subsystems might benefit from having a number of retained generations smaller than two - see OAK-7157 as an example.

      The check performed by SegmentGCOptions should be moved to SegmentNodeStoreService to prevent users from setting the number of retained generations to a an invalid value. Internal subsystems should be allowed to set the number of retained generations to a lower value, if needed.

      Attachments

        1. OAK-7158-01.diff
          6 kB
          Francesco Mari
        2. OAK-7158-02.diff
          6 kB
          Francesco Mari
        3. OAK-7158-03.diff
          9 kB
          Francesco Mari

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            frm Francesco Mari
            frm Francesco Mari
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment