Re: [PATCH 00/11] writing out a huge blob to working tree
- From
Jeff King <peff@peff.net>
- Date
- May 18, 2011, 07:50 UTC
- Message-ID
- <20110518075023.GD27482@sigill.intra.peff.net>
- In-Reply-To
- <20110518070837.GC27482@sigill.intra.peff.net>
On Wed, May 18, 2011 at 03:08:37AM -0400, Jeff King wrote:
> Part of the problem is that with the current code, all you care about is > "Is it loose, packed non-delta, or packed delta?". > [...] > The patch below implements the cheap "is_delta" check.
Hmm, sorry, this patch works well on top of cd36b7b, where I first detected the slowness, but later in the series we actually do look at the pack information in the object_info. So my patch breaks that code path horribly (I still think the concept of avoiding the revindex should still work in principle, though).
-Peff