gitaly_client: Use `ListAllLFSPointers` to retrieve all LFS pointers
What does this MR do?
The function BlobService#get_all_lfs_pointers()
is still using
ListLFSPointers
to list all LFS pointers which exist in the
repository. This function is highly inefficient given that it has to
perform a walk of the object graph to find all LFS pointers, whereas we
can just iterate over all objects directly without doing a graph walk.
Convert the function to use ListAllLFSPointers
instead to improve
performance.
Part of #337505 (closed).
Does this MR meet the acceptance criteria?
Conformity
-
I have included changelog trailers, or none are needed. (Does this MR need a changelog?) -
I have self-reviewed this MR per code review guidelines. -
This MR does not harm performance, or I have asked a reviewer to help assess the performance impact. (Merge request performance guidelines) -
I have followed the style guides. -
This change is backwards compatible across updates, or this does not apply.
Availability and Testing
-
I have added/updated tests following the Testing Guide, or it's not needed. (Consider all test levels. See the Test Planning Process.)