Skip to content

Resolve "N + 1: CommitDelta for large pushes"

Marc Shaw requested to merge 227572-n-1-commitdelta-for-large-pushes-2 into master

What does this MR do?

This issue uses the new RPC introduced that returns the paths and status's for an array of commits. This stops us calling the delta for every commit when checking the allowed endpoint.

Issue: #227572 (closed)

Rollout issue: #288827 (closed)

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #227572 (closed)

Edited by Marc Shaw

Merge request reports