Uploaded image for project: 'Confluence Data Center'
  1. Confluence Data Center
  2. CONFSERVER-20475

Improve performance of DefaultSpacePermissionManager#hasPermissionViaGroups by reversing the way permissions are checked.

    • We collect Confluence feedback from various sources, and we evaluate what we've collected when planning our product roadmap. To understand how this piece of feedback will be reviewed, see our Implementation of New Features Policy.

      When Confluence checks permissions at the group level, it retrieves all the groups a user is a member of, then looks for a matching permission against each of those groups. This performs terribly when the user is a member of a large number of groups.

      Performance would almost always be better, or at least be much more predictable if you grab all the group-level permissions for the space, then perform memberOf() checks against the user for each group. (Especially in situations like LDAP where memberOf() is really cheap, but getting all groups for a user is painfully expensive)

            [CONFSERVER-20475] Improve performance of DefaultSpacePermissionManager#hasPermissionViaGroups by reversing the way permissions are checked.

            Sen Geronimo made changes -
            Workflow Original: JAC Suggestion Workflow 4 [ 3577815 ] New: JAC Suggestion Workflow 3 [ 4327786 ]
            Katherine Yabut made changes -
            Workflow Original: JAC Suggestion Workflow 2 [ 3171337 ] New: JAC Suggestion Workflow 4 [ 3577815 ]
            Status Original: RESOLVED [ 5 ] New: Closed [ 6 ]
            Katherine Yabut made changes -
            Workflow Original: JAC Suggestion Workflow [ 3026448 ] New: JAC Suggestion Workflow 2 [ 3171337 ]
            Owen made changes -
            Workflow Original: Confluence Workflow - Public Facing v4 [ 2526466 ] New: JAC Suggestion Workflow [ 3026448 ]
            Rachel Lin (Inactive) made changes -
            Workflow Original: Confluence Workflow - Public Facing v3 [ 2272215 ] New: Confluence Workflow - Public Facing v4 [ 2526466 ]
            Katherine Yabut made changes -
            Workflow Original: Confluence Workflow - Public Facing v3 - TEMP [ 2168818 ] New: Confluence Workflow - Public Facing v3 [ 2272215 ]
            Katherine Yabut made changes -
            Workflow Original: Confluence Workflow - Public Facing v3 [ 1928237 ] New: Confluence Workflow - Public Facing v3 - TEMP [ 2168818 ]
            Katherine Yabut made changes -
            Workflow Original: Confluence Workflow - Public Facing v2 [ 1755985 ] New: Confluence Workflow - Public Facing v3 [ 1928237 ]
            Katherine Yabut made changes -
            Workflow Original: CONF Suggestion WF (TEMP) [ 1686745 ] New: Confluence Workflow - Public Facing v2 [ 1755985 ]
            Katherine Yabut made changes -
            Workflow Original: Confluence Workflow - Public Facing [ 1365892 ] New: CONF Suggestion WF (TEMP) [ 1686745 ]

              matt@atlassian.com Matt Ryall
              cmiller CharlesA
              Votes:
              7 Vote for this issue
              Watchers:
              10 Start watching this issue

                Created:
                Updated:
                Resolved: