Re: [External] Re: [PATCH 2/2] fetch-pack.c: do not declare local commits as "have" in partial repos
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 23, 2024, 16:21 UTC
- Message-ID
- <xmqqy13i49za.fsf@gitster.g>
- In-Reply-To
- <CAG1j3zHJVrpK5JZtUXFwkZgWY1-CxqET+ygpaMqo5aM-KeWaxg@mail.gmail.com>
韩仰 <hanyang.tony@bytedance.com> writes:
> In $JOB, we modified git-repack to pack everything into a giant promisor > pack if the repo is partially cloned.
I would imagine that would give you pretty much similar results as the posted patch without incurring the cost of transfering the same objects from the promisor remote.
> This basically does the same thing as > you suggested, but without the cost of constructing the object list and > removing the objects in the promisor packs.
Yup, repacking, instead of creating a new pack only to hold the objects in the gap, would be much simpler.