Re: [PATCH 0/4] optimizing upload-pack ref peeling

2012-10-04 Thread Nazri Ramliy
On Thu, Oct 4, 2012 at 5:01 PM, Ævar Arnfjörð Bjarmason wrote: > $ time (echo | ~/g/git/git-upload-pack . | pv >/dev/null) Totally off-topic, but ... Thanks for making me look up what pv is. I remember checking it out quiet sometime ago and have forgotten about it altogether. Now I can

Re: [PATCH 0/4] optimizing upload-pack ref peeling

2012-10-04 Thread Ævar Arnfjörð Bjarmason
On Thu, Oct 4, 2012 at 10:04 AM, Jeff King wrote: > On Thu, Oct 04, 2012 at 03:56:09AM -0400, Jeff King wrote: > >> [1/4]: peel_ref: use faster deref_tag_noverify >> [2/4]: peel_ref: do not return a null sha1 >> [3/4]: peel_ref: check object type before loading >> [4/4]: upload-pack: use p

Re: [PATCH 0/4] optimizing upload-pack ref peeling

2012-10-04 Thread Jeff King
On Thu, Oct 04, 2012 at 03:56:09AM -0400, Jeff King wrote: > [1/4]: peel_ref: use faster deref_tag_noverify > [2/4]: peel_ref: do not return a null sha1 > [3/4]: peel_ref: check object type before loading > [4/4]: upload-pack: use peel_ref for ref advertisements I included my own timings

[PATCH 0/4] optimizing upload-pack ref peeling

2012-10-04 Thread Jeff King
On Thu, Oct 04, 2012 at 01:54:47AM +0200, Ævar Arnfjörð Bjarmason wrote: > > Hmm. It seems like we should not need to open the tags at all. The main > > reason is to produce the "peeled" advertisement just after it. But for a > > packed ref with a modern version of git that supports the "peeled" >