Re: [PATCH 1/3] commit: allow associating auxiliary info on-demand

2013-04-14 Thread Junio C Hamano
Jeff King writes: > Thanks, this writeup makes sense to me. Obviously, > > Signed-off-by: Jeff King > > Comparing a slab index versus a util pointer, I think the differences > should be: > ... > So I think the util pointer is strictly better, except for point (2). And > the main cost is point

Re: [PATCH 1/3] commit: allow associating auxiliary info on-demand

2013-04-14 Thread Jeff King
On Sat, Apr 13, 2013 at 11:04:47PM -0700, Junio C Hamano wrote: > From: Jeff King > > The "indegree" field in the commit object is only used while sorting > a list of commits in topological order, and wasting memory otherwise. > > We would prefer to shrink the size of individual commit objects,

[PATCH 1/3] commit: allow associating auxiliary info on-demand

2013-04-13 Thread Junio C Hamano
From: Jeff King The "indegree" field in the commit object is only used while sorting a list of commits in topological order, and wasting memory otherwise. We would prefer to shrink the size of individual commit objects, which we may have to hold thousands of in-core. We could eject "indegree" fi