Re: [PATCH v2 2/2] index-pack: kill union delta_base to save memory

2015-02-28 Thread Junio C Hamano
Duy Nguyen writes: >> Probably "Not so ancient versions of Git tries to use ofs-delta >> encoding whenever possible, so it is expected that objects encoded >> using ref-delta are minority" may be closer to the truth. And that >> observation does justify why using two separate pools (one with >>

Re: [PATCH v2 2/2] index-pack: kill union delta_base to save memory

2015-02-28 Thread Duy Nguyen
On Sat, Feb 28, 2015 at 4:18 AM, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy writes: > >> Notice that with "recent" Git versions, ofs-delta objects are >> preferred over ref-delta objects and ref-delta objects have no reason >> to be present in a clone pack. > > It is true that we try to use of

Re: [PATCH v2 2/2] index-pack: kill union delta_base to save memory

2015-02-27 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > Notice that with "recent" Git versions, ofs-delta objects are > preferred over ref-delta objects and ref-delta objects have no reason > to be present in a clone pack. It is true that we try to use ofs-delta as much as possible, but where does "have no reason to be