create_pack_from_packs should check that knit compression parents are present

Bug #156091 reported by Martin Pool on 2007-10-23
2
Affects Status Importance Assigned to Milestone
Bazaar
High
Unassigned

Bug Description

The create_pack_from_packs code is used when fetching from one pack repository to another. (And also, or something similar is used, when fetching to and from knits.) This copies the compressed knit hunks for speed. It doesn't check that the knit compression parents are present, which allows situations like that of bug 155730.

Robert suggests that the check will be much more efficient if we remember the compression parents as we copy records, then check for all of them at the end.

We would have the option of giving a warning or error, or additionally copying the parents.

Martin Pool (mbp) wrote :

This isn't a problem when pulling from knits to packs as that uses the knit index to decide what to copy.

Changed in bzr:
importance: Undecided → High
status: New → Confirmed
Jelmer Vernooij (jelmer) on 2011-01-15
tags: added: packs
Jelmer Vernooij (jelmer) on 2017-11-08
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers