Implement list_refs with a single RPC (!105970) · Merge requests · GitLab.org / GitLab

James Fargher requested to merge unify_ref_names into master Dec 04, 2022

What does this MR do and why?

Behind feature flag list_refs_for_find_all_tags_branches. Rollout #383617 (closed)

While testing !102002 (merged) I noticed that Gitlab::Git::Repository#list_refs was implemented with two RPC calls, FindAllBranchNames and FindAllTagNames. Now that we use ListRefs for both of these, we can reduce the combination to a single call

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

I have evaluated the MR acceptance checklist for this MR.

Edited Dec 04, 2022 by James Fargher

Implement list_refs with a single RPC

What does this MR do and why?

MR acceptance checklist

Merge request reports