Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • lava lava
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 96
    • Issues 96
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 64
    • Merge requests 64
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • lava
  • lavalava
  • Merge requests
  • !2035

Merged
Created Mar 15, 2023 by Igor Ponomarev@igo95862Contributor

Use exists() instead of count() for is_permission_restricted

  • Overview 0
  • Commits 1
  • Changes 1

There is no point in counting all permissions if the final count will only be compared to zero.

Instead of use exists() which returns True as soon as first permission encountered.

Gives +10% performance boost even on small permission tables and much larger boost for large permission tables.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: is-permission-restricted-use-exists