The source project of this merge request has been removed.
When viewing a commit, do not go to the blob for the file path
What does this MR do?
This defers a slow Gitaly call when looking at a single commit. We already have the filename in memory, so can use it directly here.
It doesn't completely solve the issue - we still load the blobs to check whether a file is binary or not - but we don't haver to load as many once this is merged.
Screenshots
GetBlobs RPC before
GetBlobs RPC after
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation created/updated or follow-up review issue created -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Related to #31118 (closed)
Edited by Bob Van Landuyt