Skip to content

Plumbing to allow blames to be limited or paginated

Nick Thomas requested to merge 225174-paginated-git-blame into master

In #225174 (closed) , we discover that the blame for https://gitlab.com/ddevault/linux/-/blob/master/MAINTAINERS is too slow at the Gitaly level. Adding a limit is discussed, but making just the first 1,000 lines (for instance) of the blame output visible is not useful for such a file.

The Gitaly support is in gitaly!4433 (merged) . This MR adds the plumbing necessary to use this new argument in GitLab, including a view-level change (to account for the possibility that blames may no longer start at 1), but never actually sets the new range parameter anywhere but in tests, so should be a no-op.

I envision working on adding support to the REST API next: !84629 (merged)

Related to #225174 (closed)

Edited by Nick Thomas

Merge request reports