Skip to content

Exclude the total headers from the commits API

Bob Van Landuyt requested to merge bvl-commits-api-exclude-count into master

What does this MR do?

This removes the CountCommits gitaly call that was part of the commits API. A consequence of this is that we had to remove the X-Total and X-Total-Pages pagination headers. We already do this for some ActiveRecord::Relation-pagers.

For now, this change is behind the api_commits_without_count feature flag.

This could help with https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/470

Feature flag roll-out: #254994 (closed)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Bob Van Landuyt

Merge request reports