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

Move tree type to permission implementation

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.16
    • core
    • None

    Description

      As suggested by jukka on the mailing list we may consider removing the tree type from the ImmutableTree and ImmutableRoot and just keep it inside the permission evaluation code.

      pro: simplify ImmutableTree

      con: for PermissionProvider#isGranted and PermissionProvider#hasPrivilege the treepermission object is not present and retrieving the type always will walk up the hierarchy to retrieve the type of the parent with the associated performance impact.

      Attachments

        1. OAK-1386.patch
          36 kB
          Angela Schreiber

        Activity

          People

            angela Angela Schreiber
            angela Angela Schreiber
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: