Re: [PATCH v10 0/5] fetch: avoid fetching every branch of a new remote in a shallow repo

[Junio C Hamano <gitster@pobox.com>]

"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:

> Avoid fetching every branch of a new remote in a shallow repo.
>
> Changes in v10:
>
>  * Resolve merge conflict after master integrated
>    ch/fetch-followremotehead-lazy-validation.

> Harald Nordgren (5):
>   fetch: add remote.<name>.refmap
>   fetch: extract collect_upstream_from_remote() helper
>   fetch: infer branches to fetch from a refmap-only remote
>   remote: add "git remote add --limited-fetch"
>   remote: default to --limited-fetch in a shallow repository
>
>  Documentation/config/remote.adoc |   8 +++
>  Documentation/fetch-options.adoc |   5 ++
>  Documentation/git-remote.adoc    |  14 +++-
>  builtin/fetch.c                  |  92 ++++++++++++++++++++----
>  builtin/remote.c                 |  31 ++++++--
>  remote.c                         |  51 ++++++++++++-
>  remote.h                         |  19 +++++
>  t/meson.build                    |   1 +
>  t/t5505-remote.sh                |  76 ++++++++++++++++++++
>  t/t5510-fetch.sh                 |  63 ++++++++++++++++
>  t/t5586-fetch-refmap.sh          | 119 +++++++++++++++++++++++++++++++
>  11 files changed, 456 insertions(+), 23 deletions(-)
>  create mode 100755 t/t5586-fetch-refmap.sh

This topic seeems fairly well done, without seeing any additional
comments in the last two rounds, but I am highly disturbed that I
was practically the only person who reviewd its later iterations.
Because I am not a target audience of the feature, I would not be
surprised if I am missing something large.

I would very much appreciate if somebody else can give the series
one more read and Ack them, so that we can move it forward.

Thanks.