Re: [PATCH 4/6] introduce a commit metapack
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Jan 30, 2013, 14:16 UTC
- Message-ID
- <CACsJy8Bqg6knVtddwaGSqtiXzVDgbO1JjbFOPMbF_RqrxM2rFQ@mail.gmail.com>
- In-Reply-To
- <20130130135607.GA23154@lanh>
On Wed, Jan 30, 2013 at 8:56 PM, Duy Nguyen <pclouds@gmail.com> wrote:
> However, performance seems to suffer too. Maybe I do more lookups than > necessary, I don't know.
Yes, I should have stored the position in the sha-1 <-> offset map instead of the position of the object in .pack file. Even so, performance does not improve.
> I should probably measure the cost of revindex separately.
And the cost of create_pack_revindex() is 0.6 sec :-(
Perhaps we could store abbrev sha-1 instead of full sha-1. Nice space/time trade-off.
-- Duy