Re: [PATCHv4 08/12] Teach the notes lookup code to parse notes trees with various fanout schemes
- From
Jeff King <peff@peff.net>
- Date
- Aug 27, 2009, 23:39 UTC
- Message-ID
- <20090827233900.GA7347@coredump.intra.peff.net>
- In-Reply-To
- <200908280103.06015.johan@herland.net>
On Fri, Aug 28, 2009 at 01:03:05AM +0200, Johan Herland wrote:
Show 8 quoted lines
> Agreed. I'm starting to come around to the idea of storing them in subtrees > based on commit dates. For one, you don't have multiple notes for one commit > in the same notes tree. Also, the common-case access pattern seems tempting. > > Dscho: Were there other problems with the date-based approach other than not > supporting notes on trees and blobs? > > If not, I'll start preparing another series with the date-based approach.
Would you ever want to load a note for a commit when you did not have that commit present (in which case you would not know its date)?
-Peff