Skip to content

Fix Gitaly duration measurements

Stan Hu requested to merge sh-gitaly-duration-measurement into master

In many cases, we were only measuring the time to return from the Gitaly RPC, not the total execution time of the RPC itself for streaming responses. To handle this, we wrap the RPC in a new method, GitalyClient.streaming_call that yields a response and consumes it.

Only CommitService has been updated to use this new measurement. Other services should be updated in subsequent merge requests.

Relates to #30334 (closed)

Edited by Stan Hu

Merge request reports