Skip to content

Remove Rugged::Walk implementation of #log

Jacob Vosmaer requested to merge repository-log-disable-walk into master

What does this MR do?

This is the second shot at removing this code, after https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16525 got reverted in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16676.

We had to revert at the time because we were diverting requests to a code path that did not handle alternate object directories well in gitaly-ruby. This has been resolved in Gitaly 0.76.0, via https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16715.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by Jacob Vosmaer

Merge request reports

Loading