Change response of ListRemotesResponse from single message to stream (!262) · Merge requests · GitLab.org / gitaly-proto · GitLab

This is an archived project. Repository and other project resources are read-only.

Michael Leopard requested to merge stream-list-remotes into master Jan 29, 2019

what is changing
- List remotes is being changed to stream git remote information for better performance based on this comment
why we're changing it
- We're changing it in favor of better performance based on this comment from this merge request
why this change is OK from a forwards/backwards compatibility viewpoint (nothing/nobody is calling this RPC yet)
- Nothing is using this RPC yet. This is being implemented by an open merge request

Edited Jan 29, 2019 by Jacob Vosmaer