Re: cb/clear-quarantine-early-on-all-ref-update-errors

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 4, 2022, 16:45 UTC
Message-ID: <xmqqa6f6oa2b.fsf@gitster.g>
In-Reply-To: <220204.86mtj63jg5.gmgdl@evledraar.gmail.com>

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

Show 17 quoted lines

> On Thu, Feb 03 2022, Junio C Hamano wrote:
>
>> * cb/clear-quarantine-early-on-all-ref-update-errors (2022-02-01) 1 commit
>>  - receive-pack: purge temporary data if no command is ready to run
>>
>>  Check if "receive-pack" will do any ref updates (various conditions
>>  could reject a push) before received objects are taken out of the
>>  temporary directory used for quarantine purposes, so that a push
>>  that is known-to-fail will not leave crufts that a future "gc"
>>  needs to clean up.
>>
>>  Will merge to 'next'?
>>  source: <20220129063538.24038-1-bojun.cbj@gmail.com>
>
> Ah, I see it's even in the same function as
> rj/receive-pack-abort-upon-disconnect. That's a very well reasoned &
> sensible isolated change. I think it's ready for "next".

I have a small comment on possible follow-on work, but that wouldn't be a blocker. This seems to be a pure improvement.

Thanks.