Skip to content

Implement fix for n+1 issue on `flatten_tree` helper

Alejandro Rodríguez requested to merge gitaly-tree-entries-fix into master

What does this MR do?

Use the new field flat_path that Gitaly populates to skip calculating it when rendering the views, which is an N+1 operation

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

No

Why was this MR needed?

CommitService.GetTreeEntries RPC was being called in a n+1 fashion, which is no good.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes gitaly#530 (closed)

Merge request reports