Re: [PATCH v2 0/2] Document two partial clone bugs, fix one
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 22, 2020, 17:25 UTC
- Message-ID
- <xmqqy2su5yal.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <pull.556.v2.git.1582321648.gitgitgadget@gmail.com>
"Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 20 quoted lines
> While playing with partial clone, I discovered a few bugs and document them > with tests in patch 1. One seems to be a server-side bug that happens in a > somewhat rare situation, but not terribly unlikely. The other is a > client-side bug that leads to quadratic amounts of data transfer; I fix this > bug in patch 2. > > UPDATES in V2: > > * Added "|| return 1" inside the for loops. > > > * Added an in-test comment about the test ordering. > > > * Required protocol.version=2 in the tags test due to the bisect Junio > performed. > > > * Updated the commit message via Jonathan Tan's suggestion. >
Now this can safely be queued directly on v2.25.0, I'll rebase it (earlyer I queued it after the merge to make protocol v2 the default).
Thanks.