From: Jeff King <peff@peff.net>
Date: Mon, 20 Jul 2020 17:42:03 GMT
Subject: Re: [PATCH v2] upload-pack: do not lazy-fetch "have" objects
Message-ID: <20200720174203.GB2046890@coredump.intra.peff.net>
In-Reply-To: <20200716180950.2852753-1-jonathantanmy@google.com>

On Thu, Jul 16, 2020 at 11:09:50AM -0700, Jonathan Tan wrote:

> Differences from v1: OBJECT_INFO_QUICK is also used wherever
> OBJECT_INFO_SKIP_FETCH_OBJECT is added.
> 
> Using OBJECT_INFO_QUICK makes sense to me, so here's an updated patch.

Yeah, I think this is reasonable, and the patch looks good to me.

-Peff