Skip to content

Fix MR diff file counts for some historic data

Nick Thomas requested to merge 246489-fix-files-count-for-large-mr-diffs into master

What does this MR do?

Some old diffs in the database for GitLab.com are much bigger than current diff limits permit - one example has around 45K records. The files_count value isn't that important, so we can work around this without more schema changes by using (signed) SMALLINT_MAX as a sentinel value.

Ref: #246489 (closed)

Screenshots

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
Edited by Markus Koller

Merge request reports