On Wed, Jun 26, 2013 at 6:53 PM, Colby Ranger <cranger@xxxxxxxxxx> wrote: >> + Generating this reverse index at runtime is **not** free (around 900ms >> + generation time for a repository like `torvalds/linux`), and once again, >> + this generation time needs to happen every time `pack-objects` is >> + spawned. 900ms is fishy. Creating the revidx should not take that long. But if it is... > If generating the reverse index is expensive, it is probably > worthwhile to create a ".revidx" or extend the ".idx" with the > information sorted by offset. Colby is probably right that a cached copy of the revidx would help. Or update the .idx format to have two additional sections that stores the length of each packed object and the delta base of each packed object, allowing pack-objects to avoid creating the revidx. This would be an additional ~8 bytes per object, so ~19.8M for the Linux kernel (given ~2.6M objects). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html