From: Thomas Rast
Date: Sat, 07 Dec 2013 16:59:28 GMT
Subject: Re: [PATCH v3 21/21] pack-bitmap: implement optional name_hash cache
Message-ID: <87wqjgha1b.fsf@linux-1gf2.Speedport_W723_V_Typ_A_1_00_098>
In-Reply-To: <20131114124855.GB11612@sigill.intra.peff.net>
Jeff King writes:
> Test origin/master HEAD^ HEAD
> -------------------------------------------------------------------------------------------------
> 5310.2: repack to disk 36.81(37.82+1.43) 47.70(48.74+1.41) +29.6% 47.75(48.70+1.51) +29.7%
> 5310.3: simulated clone 30.78(29.70+2.14) 1.08(0.97+0.10) -96.5% 1.07(0.94+0.12) -96.5%
> 5310.4: simulated fetch 3.16(6.10+0.08) 3.54(10.65+0.06) +12.0% 1.70(3.07+0.06) -46.2%
> 5310.6: partial bitmap 36.76(43.19+1.81) 6.71(11.25+0.76) -81.7% 4.08(6.26+0.46) -88.9%
>
> You can see that the time spent on an incremental fetch goes
> down, as our delta heuristics are able to do their work.
> And we save time on the partial bitmap clone for the same
> reason.
The time now goes down across the board compared to master. Good job!
> Signed-off-by: Vicent Marti
> Signed-off-by: Jeff King
Reviewed-by: Thomas Rast
--
Thomas Rast
tr@thomasrast.ch