Friday, June 3, 2011

Cycle 1.7.6, day #5 of week #6

Started to revisit a stalled topic to get rid of inefficient verify-pack that walks the objects in a pack in their object name order (which breaks the locality in the pack completely). The index-pack code has to internally perform the same task, and it does take object locality into account, so there is no reason not to take advantage of it and share the code. This will in the future help us to maintain one single code for packfile verification, which needs to be tweaked when I introduce a narrow clone properly. One of the important things pack verifier needs to be aware of is that it will no longer be an error if objects that are referenced by an object in the pack do not exist in the repository (when they are outside the narrowed domain).
Sent a four-patch series for review to the list.

Commented on a dozen topics.
Queued 2 patches from 2 people.

No comments: