Re: [PATCH v2 0/2] Restrict when prefetcing occurs
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Apr 6, 2020, 11:44 UTC
- Message-ID
- <7de2f54b-8704-a0e1-12aa-0ca9d3d70f6f@gmail.com>
- In-Reply-To
- <xmqq369l3a4a.fsf@gitster.c.googlers.com>
On 4/2/2020 4:28 PM, Junio C Hamano wrote:
Show 5 quoted lines
> I notice that a439b4ef (diff: skip batch object download when > possible, 2020-03-30) by Garima seems to aim for something similar. > > I'll for now keep both topics with conflict resolution, but it may > make sense for you two to compare notes.
I pointed this out in [1]. I think the right thing to do is for Garima's/my patch to rely on Jonathan's change. The commit needs to be modified, not simply ejected, but it could be separated from the rest of Garima's series. It is only a performance fix for normal clones, but is critical for partial clones.
Garima: do you think it would be easy to remove that patch if/when you do a v4 and I can make a new series based on yours and Jonathan's with the rename setting?
[1] https://lore.kernel.org/git/b956528c-412b-2f38-bd90-1fa2ae4b8604@gmail.com/ Thanks, -Stolee