~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/reconcile.py

  • Committer: John Arbash Meinel
  • Date: 2006-07-12 02:49:48 UTC
  • mto: (1946.2.6 reduce-knit-churn)
  • mto: This revision was merged to the branch mainline in revision 1855.
  • Revision ID: john@arbash-meinel.com-20060712024948-162ebe6b4b9985bf
Restore logging of full traceback. Helps with debugging tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006 Canonical Ltd
 
1
# (C) 2005, 2006 Canonical Limited.
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""Reconcilers are able to fix some potential data errors in a branch."""
18
18
 
19
19
 
20
 
__all__ = [
21
 
    'KnitReconciler',
22
 
    'PackReconciler',
23
 
    'reconcile',
24
 
    'Reconciler',
25
 
    'RepoReconciler',
26
 
    ]
27
 
 
28
 
 
29
 
from bzrlib import (
30
 
    errors,
31
 
    ui,
32
 
    repository,
33
 
    repofmt,
34
 
    )
35
 
from bzrlib.trace import mutter, note
36
 
from bzrlib.tsort import topo_sort
37
 
from bzrlib.versionedfile import AdapterFactory, FulltextContentFactory
 
20
__all__ = ['reconcile', 'Reconciler', 'RepoReconciler', 'KnitReconciler']
 
21
 
 
22
 
 
23
from bzrlib import ui
 
24
from bzrlib.trace import mutter
 
25
from bzrlib.tsort import TopoSorter
38
26
 
39
27
 
40
28
def reconcile(dir, other=None):
62
50
 
63
51
    def reconcile(self):
64
52
        """Perform reconciliation.
65
 
 
 
53
        
66
54
        After reconciliation the following attributes document found issues:
67
55
        inconsistent_parents: The number of revisions in the repository whose
68
56
                              ancestry was being reported incorrectly.
69
57
        garbage_inventories: The number of inventory objects without revisions
70
58
                             that were garbage collected.
71
 
        fixed_branch_history: None if there was no branch, False if the branch
72
 
                              history was correct, True if the branch history
73
 
                              needed to be re-normalized.
74
59
        """
75
60
        self.pb = ui.ui_factory.nested_progress_bar()
76
61
        try:
80
65
 
81
66
    def _reconcile(self):
82
67
        """Helper function for performing reconciliation."""
83
 
        self._reconcile_branch()
84
 
        self._reconcile_repository()
85
 
 
86
 
    def _reconcile_branch(self):
87
 
        try:
88
 
            self.branch = self.bzrdir.open_branch()
89
 
        except errors.NotBranchError:
90
 
            # Nothing to check here
91
 
            self.fixed_branch_history = None
92
 
            return
93
 
        self.pb.note('Reconciling branch %s',
94
 
                     self.branch.base)
95
 
        branch_reconciler = self.branch.reconcile(thorough=True)
96
 
        self.fixed_branch_history = branch_reconciler.fixed_history
97
 
 
98
 
    def _reconcile_repository(self):
99
68
        self.repo = self.bzrdir.find_repository()
100
69
        self.pb.note('Reconciling repository %s',
101
70
                     self.repo.bzrdir.root_transport.base)
102
 
        self.pb.update("Reconciling repository", 0, 1)
103
71
        repo_reconciler = self.repo.reconcile(thorough=True)
104
72
        self.inconsistent_parents = repo_reconciler.inconsistent_parents
105
73
        self.garbage_inventories = repo_reconciler.garbage_inventories
106
 
        if repo_reconciler.aborted:
107
 
            self.pb.note(
108
 
                'Reconcile aborted: revision index has inconsistent parents.')
109
 
            self.pb.note(
110
 
                'Run "bzr check" for more details.')
111
 
        else:
112
 
            self.pb.note('Reconciliation complete.')
113
 
 
114
 
 
115
 
class BranchReconciler(object):
116
 
    """Reconciler that works on a branch."""
117
 
 
118
 
    def __init__(self, a_branch, thorough=False):
119
 
        self.fixed_history = None
120
 
        self.thorough = thorough
121
 
        self.branch = a_branch
122
 
 
123
 
    def reconcile(self):
124
 
        self.branch.lock_write()
125
 
        try:
126
 
            self.pb = ui.ui_factory.nested_progress_bar()
127
 
            try:
128
 
                self._reconcile_steps()
129
 
            finally:
130
 
                self.pb.finished()
131
 
        finally:
132
 
            self.branch.unlock()
133
 
 
134
 
    def _reconcile_steps(self):
135
 
        self._reconcile_revision_history()
136
 
 
137
 
    def _reconcile_revision_history(self):
138
 
        repo = self.branch.repository
139
 
        last_revno, last_revision_id = self.branch.last_revision_info()
140
 
        real_history = []
141
 
        try:
142
 
            for revid in repo.iter_reverse_revision_history(
143
 
                    last_revision_id):
144
 
                real_history.append(revid)
145
 
        except errors.RevisionNotPresent:
146
 
            pass # Hit a ghost left hand parent
147
 
        real_history.reverse()
148
 
        if last_revno != len(real_history):
149
 
            self.fixed_history = True
150
 
            # Technically for Branch5 formats, it is more efficient to use
151
 
            # set_revision_history, as this will regenerate it again.
152
 
            # Not really worth a whole BranchReconciler class just for this,
153
 
            # though.
154
 
            self.pb.note('Fixing last revision info %s => %s',
155
 
                         last_revno, len(real_history))
156
 
            self.branch.set_last_revision_info(len(real_history),
157
 
                                               last_revision_id)
158
 
        else:
159
 
            self.fixed_history = False
160
 
            self.pb.note('revision_history ok.')
 
74
        self.pb.note('Reconciliation complete.')
161
75
 
162
76
 
163
77
class RepoReconciler(object):
164
78
    """Reconciler that reconciles a repository.
165
79
 
166
 
    The goal of repository reconciliation is to make any derived data
167
 
    consistent with the core data committed by a user. This can involve
168
 
    reindexing, or removing unreferenced data if that can interfere with
169
 
    queries in a given repository.
170
 
 
171
80
    Currently this consists of an inventory reweave with revision cross-checks.
172
81
    """
173
82
 
180
89
        """
181
90
        self.garbage_inventories = 0
182
91
        self.inconsistent_parents = 0
183
 
        self.aborted = False
184
92
        self.repo = repo
185
93
        self.thorough = thorough
186
94
 
187
95
    def reconcile(self):
188
96
        """Perform reconciliation.
189
 
 
 
97
        
190
98
        After reconciliation the following attributes document found issues:
191
99
        inconsistent_parents: The number of revisions in the repository whose
192
100
                              ancestry was being reported incorrectly.
209
117
 
210
118
    def _reweave_inventory(self):
211
119
        """Regenerate the inventory weave for the repository from scratch.
212
 
 
213
 
        This is a smart function: it will only do the reweave if doing it
 
120
        
 
121
        This is a smart function: it will only do the reweave if doing it 
214
122
        will correct data issues. The self.thorough flag controls whether
215
123
        only data-loss causing issues (!self.thorough) or all issues
216
124
        (self.thorough) are treated as requiring the reweave.
218
126
        # local because needing to know about WeaveFile is a wart we want to hide
219
127
        from bzrlib.weave import WeaveFile, Weave
220
128
        transaction = self.repo.get_transaction()
221
 
        self.pb.update('Reading inventory data')
222
 
        self.inventory = self.repo.inventories
223
 
        self.revisions = self.repo.revisions
 
129
        self.pb.update('Reading inventory data.')
 
130
        self.inventory = self.repo.get_inventory_weave()
224
131
        # the total set of revisions to process
225
 
        self.pending = set([key[-1] for key in self.revisions.keys()])
 
132
        self.pending = set([rev_id for rev_id in self.repo._revision_store.all_revision_ids(transaction)])
226
133
 
227
134
        # mapping from revision_id to parents
228
135
        self._rev_graph = {}
234
141
            # put a revision into the graph.
235
142
            self._graph_revision(rev_id)
236
143
        self._check_garbage_inventories()
237
 
        # if there are no inconsistent_parents and
 
144
        # if there are no inconsistent_parents and 
238
145
        # (no garbage inventories or we are not doing a thorough check)
239
 
        if (not self.inconsistent_parents and
 
146
        if (not self.inconsistent_parents and 
240
147
            (not self.garbage_inventories or not self.thorough)):
241
148
            self.pb.note('Inventory ok.')
242
149
            return
243
 
        self.pb.update('Backing up inventory', 0, 0)
244
 
        self.repo._backup_inventory()
245
 
        self.pb.note('Backup inventory created.')
246
 
        new_inventories = self.repo._temp_inventories()
 
150
        self.pb.update('Backing up inventory...', 0, 0)
 
151
        self.repo.control_weaves.copy(self.inventory, 'inventory.backup', self.repo.get_transaction())
 
152
        self.pb.note('Backup Inventory created.')
 
153
        # asking for '' should never return a non-empty weave
 
154
        new_inventory_vf = self.repo.control_weaves.get_empty('inventory.new',
 
155
            self.repo.get_transaction())
247
156
 
248
157
        # we have topological order of revisions and non ghost parents ready.
249
158
        self._setup_steps(len(self._rev_graph))
250
 
        revision_keys = [(rev_id,) for rev_id in topo_sort(self._rev_graph)]
251
 
        stream = self._change_inv_parents(
252
 
            self.inventory.get_record_stream(revision_keys, 'unordered', True),
253
 
            self._new_inv_parents,
254
 
            set(revision_keys))
255
 
        new_inventories.insert_record_stream(stream)
256
 
        # if this worked, the set of new_inventories.keys should equal
 
159
        for rev_id in TopoSorter(self._rev_graph.items()).iter_topo_order():
 
160
            parents = self._rev_graph[rev_id]
 
161
            # double check this really is in topological order.
 
162
            unavailable = [p for p in parents if p not in new_inventory_vf]
 
163
            assert len(unavailable) == 0
 
164
            # this entry has all the non ghost parents in the inventory
 
165
            # file already.
 
166
            self._reweave_step('adding inventories')
 
167
            if isinstance(new_inventory_vf, WeaveFile):
 
168
                # It's really a WeaveFile, but we call straight into the
 
169
                # Weave's add method to disable the auto-write-out behaviour.
 
170
                # This is done to avoid a revision_count * time-to-write additional overhead on 
 
171
                # reconcile.
 
172
                new_inventory_vf._check_write_ok()
 
173
                Weave._add_lines(new_inventory_vf, rev_id, parents, self.inventory.get_lines(rev_id),
 
174
                                 None)
 
175
            else:
 
176
                new_inventory_vf.add_lines(rev_id, parents, self.inventory.get_lines(rev_id))
 
177
 
 
178
        if isinstance(new_inventory_vf, WeaveFile):
 
179
            new_inventory_vf._save()
 
180
        # if this worked, the set of new_inventory_vf.names should equal
257
181
        # self.pending
258
 
        if not (set(new_inventories.keys()) ==
259
 
            set([(revid,) for revid in self.pending])):
260
 
            raise AssertionError()
 
182
        assert set(new_inventory_vf.versions()) == self.pending
261
183
        self.pb.update('Writing weave')
262
 
        self.repo._activate_new_inventory()
 
184
        self.repo.control_weaves.copy(new_inventory_vf, 'inventory', self.repo.get_transaction())
 
185
        self.repo.control_weaves.delete('inventory.new', self.repo.get_transaction())
263
186
        self.inventory = None
264
187
        self.pb.note('Inventory regenerated.')
265
188
 
266
 
    def _new_inv_parents(self, revision_key):
267
 
        """Lookup ghost-filtered parents for revision_key."""
268
 
        # Use the filtered ghostless parents list:
269
 
        return tuple([(revid,) for revid in self._rev_graph[revision_key[-1]]])
270
 
 
271
 
    def _change_inv_parents(self, stream, get_parents, all_revision_keys):
272
 
        """Adapt a record stream to reconcile the parents."""
273
 
        for record in stream:
274
 
            wanted_parents = get_parents(record.key)
275
 
            if wanted_parents and wanted_parents[0] not in all_revision_keys:
276
 
                # The check for the left most parent only handles knit
277
 
                # compressors, but this code only applies to knit and weave
278
 
                # repositories anyway.
279
 
                bytes = record.get_bytes_as('fulltext')
280
 
                yield FulltextContentFactory(record.key, wanted_parents, record.sha1, bytes)
281
 
            else:
282
 
                adapted_record = AdapterFactory(record.key, wanted_parents, record)
283
 
                yield adapted_record
284
 
            self._reweave_step('adding inventories')
285
 
 
286
189
    def _setup_steps(self, new_total):
287
190
        """Setup the markers we need to control the progress bar."""
288
191
        self.total = new_total
294
197
        # analyse revision id rev_id and put it in the stack.
295
198
        self._reweave_step('loading revisions')
296
199
        rev = self.repo.get_revision_reconcile(rev_id)
 
200
        assert rev.revision_id == rev_id
297
201
        parents = []
298
202
        for parent in rev.parent_ids:
299
203
            if self._parent_is_available(parent):
300
204
                parents.append(parent)
301
205
            else:
302
206
                mutter('found ghost %s', parent)
303
 
        self._rev_graph[rev_id] = parents
 
207
        self._rev_graph[rev_id] = parents   
 
208
        if self._parents_are_inconsistent(rev_id, parents):
 
209
            self.inconsistent_parents += 1
 
210
            mutter('Inconsistent inventory parents: id {%s} '
 
211
                   'inventory claims %r, '
 
212
                   'available parents are %r, '
 
213
                   'unavailable parents are %r',
 
214
                   rev_id, 
 
215
                   set(self.inventory.get_parents(rev_id)),
 
216
                   set(parents),
 
217
                   set(rev.parent_ids).difference(set(parents)))
 
218
 
 
219
    def _parents_are_inconsistent(self, rev_id, parents):
 
220
        """Return True if the parents list of rev_id does not match the weave.
 
221
 
 
222
        This detects inconsistencies based on the self.thorough value:
 
223
        if thorough is on, the first parent value is checked as well as ghost
 
224
        differences.
 
225
        Otherwise only the ghost differences are evaluated.
 
226
        """
 
227
        weave_parents = self.inventory.get_parents(rev_id)
 
228
        weave_missing_old_ghosts = set(weave_parents) != set(parents)
 
229
        first_parent_is_wrong = (
 
230
            len(weave_parents) and len(parents) and
 
231
            parents[0] != weave_parents[0])
 
232
        if self.thorough:
 
233
            return weave_missing_old_ghosts or first_parent_is_wrong
 
234
        else:
 
235
            return weave_missing_old_ghosts
304
236
 
305
237
    def _check_garbage_inventories(self):
306
238
        """Check for garbage inventories which we cannot trust
310
242
        """
311
243
        if not self.thorough:
312
244
            return
313
 
        inventories = set(self.inventory.keys())
314
 
        revisions = set(self.revisions.keys())
 
245
        inventories = set(self.inventory.versions())
 
246
        revisions = set(self._rev_graph.keys())
315
247
        garbage = inventories.difference(revisions)
316
248
        self.garbage_inventories = len(garbage)
317
 
        for revision_key in garbage:
318
 
            mutter('Garbage inventory {%s} found.', revision_key[-1])
 
249
        for revision_id in garbage:
 
250
            mutter('Garbage inventory {%s} found.', revision_id)
319
251
 
320
252
    def _parent_is_available(self, parent):
321
253
        """True if parent is a fully available revision
323
255
        A fully available revision has a inventory and a revision object in the
324
256
        repository.
325
257
        """
326
 
        if parent in self._rev_graph:
327
 
            return True
328
 
        inv_present = (1 == len(self.inventory.get_parent_map([(parent,)])))
329
 
        return (inv_present and self.repo.has_revision(parent))
 
258
        return (parent in self._rev_graph or 
 
259
                (parent in self.inventory and self.repo.has_revision(parent)))
330
260
 
331
261
    def _reweave_step(self, message):
332
262
        """Mark a single step of regeneration complete."""
337
267
class KnitReconciler(RepoReconciler):
338
268
    """Reconciler that reconciles a knit format repository.
339
269
 
340
 
    This will detect garbage inventories and remove them in thorough mode.
 
270
    This will detect garbage inventories and remove them.
 
271
 
 
272
    Inconsistent parentage is checked for in the revision weave.
341
273
    """
342
274
 
343
275
    def _reconcile_steps(self):
344
276
        """Perform the steps to reconcile this repository."""
345
277
        if self.thorough:
346
 
            try:
347
 
                self._load_indexes()
348
 
            except errors.BzrCheckError:
349
 
                self.aborted = True
350
 
                return
 
278
            self._load_indexes()
351
279
            # knits never suffer this
352
280
            self._gc_inventory()
353
 
            self._fix_text_parents()
354
281
 
355
282
    def _load_indexes(self):
356
283
        """Load indexes for the reconciliation."""
357
284
        self.transaction = self.repo.get_transaction()
358
 
        self.pb.update('Reading indexes', 0, 2)
359
 
        self.inventory = self.repo.inventories
360
 
        self.pb.update('Reading indexes', 1, 2)
361
 
        self.repo._check_for_inconsistent_revision_parents()
362
 
        self.revisions = self.repo.revisions
363
 
        self.pb.update('Reading indexes', 2, 2)
 
285
        self.pb.update('Reading indexes.', 0, 2)
 
286
        self.inventory = self.repo.get_inventory_weave()
 
287
        self.pb.update('Reading indexes.', 1, 2)
 
288
        self.revisions = self.repo._revision_store.get_revision_file(self.transaction)
 
289
        self.pb.update('Reading indexes.', 2, 2)
364
290
 
365
291
    def _gc_inventory(self):
366
292
        """Remove inventories that are not referenced from the revision store."""
367
 
        self.pb.update('Checking unused inventories', 0, 1)
 
293
        self.pb.update('Checking unused inventories.', 0, 1)
368
294
        self._check_garbage_inventories()
369
 
        self.pb.update('Checking unused inventories', 1, 3)
 
295
        self.pb.update('Checking unused inventories.', 1, 3)
370
296
        if not self.garbage_inventories:
371
297
            self.pb.note('Inventory ok.')
372
298
            return
373
 
        self.pb.update('Backing up inventory', 0, 0)
374
 
        self.repo._backup_inventory()
375
 
        self.pb.note('Backup Inventory created')
 
299
        self.pb.update('Backing up inventory...', 0, 0)
 
300
        self.repo.control_weaves.copy(self.inventory, 'inventory.backup', self.transaction)
 
301
        self.pb.note('Backup Inventory created.')
376
302
        # asking for '' should never return a non-empty weave
377
 
        new_inventories = self.repo._temp_inventories()
 
303
        new_inventory_vf = self.repo.control_weaves.get_empty('inventory.new',
 
304
            self.transaction)
 
305
 
378
306
        # we have topological order of revisions and non ghost parents ready.
379
 
        graph = self.revisions.get_parent_map(self.revisions.keys())
380
 
        revision_keys = topo_sort(graph)
381
 
        revision_ids = [key[-1] for key in revision_keys]
382
 
        self._setup_steps(len(revision_keys))
383
 
        stream = self._change_inv_parents(
384
 
            self.inventory.get_record_stream(revision_keys, 'unordered', True),
385
 
            graph.__getitem__,
386
 
            set(revision_keys))
387
 
        new_inventories.insert_record_stream(stream)
 
307
        self._setup_steps(len(self.revisions))
 
308
        for rev_id in TopoSorter(self.revisions.get_graph().items()).iter_topo_order():
 
309
            parents = self.revisions.get_parents(rev_id)
 
310
            # double check this really is in topological order.
 
311
            unavailable = [p for p in parents if p not in new_inventory_vf]
 
312
            assert len(unavailable) == 0
 
313
            # this entry has all the non ghost parents in the inventory
 
314
            # file already.
 
315
            self._reweave_step('adding inventories')
 
316
            # ugly but needed, weaves are just way tooooo slow else.
 
317
            new_inventory_vf.add_lines(rev_id, parents, self.inventory.get_lines(rev_id))
 
318
 
388
319
        # if this worked, the set of new_inventory_vf.names should equal
389
 
        # the revisionds list
390
 
        if not(set(new_inventories.keys()) == set(revision_keys)):
391
 
            raise AssertionError()
 
320
        # self.pending
 
321
        assert set(new_inventory_vf.versions()) == set(self.revisions.versions())
392
322
        self.pb.update('Writing weave')
393
 
        self.repo._activate_new_inventory()
 
323
        self.repo.control_weaves.copy(new_inventory_vf, 'inventory', self.transaction)
 
324
        self.repo.control_weaves.delete('inventory.new', self.transaction)
394
325
        self.inventory = None
395
326
        self.pb.note('Inventory regenerated.')
396
327
 
397
 
    def _fix_text_parents(self):
398
 
        """Fix bad versionedfile parent entries.
399
 
 
400
 
        It is possible for the parents entry in a versionedfile entry to be
401
 
        inconsistent with the values in the revision and inventory.
402
 
 
403
 
        This method finds entries with such inconsistencies, corrects their
404
 
        parent lists, and replaces the versionedfile with a corrected version.
405
 
        """
406
 
        transaction = self.repo.get_transaction()
407
 
        versions = [key[-1] for key in self.revisions.keys()]
408
 
        mutter('Prepopulating revision text cache with %d revisions',
409
 
                len(versions))
410
 
        vf_checker = self.repo._get_versioned_file_checker()
411
 
        bad_parents, unused_versions = vf_checker.check_file_version_parents(
412
 
            self.repo.texts, self.pb)
413
 
        text_index = vf_checker.text_index
414
 
        per_id_bad_parents = {}
415
 
        for key in unused_versions:
416
 
            # Ensure that every file with unused versions gets rewritten.
417
 
            # NB: This is really not needed, reconcile != pack.
418
 
            per_id_bad_parents[key[0]] = {}
419
 
        # Generate per-knit/weave data.
420
 
        for key, details in bad_parents.iteritems():
421
 
            file_id = key[0]
422
 
            rev_id = key[1]
423
 
            knit_parents = tuple([parent[-1] for parent in details[0]])
424
 
            correct_parents = tuple([parent[-1] for parent in details[1]])
425
 
            file_details = per_id_bad_parents.setdefault(file_id, {})
426
 
            file_details[rev_id] = (knit_parents, correct_parents)
427
 
        file_id_versions = {}
428
 
        for text_key in text_index:
429
 
            versions_list = file_id_versions.setdefault(text_key[0], [])
430
 
            versions_list.append(text_key[1])
431
 
        # Do the reconcile of individual weaves.
432
 
        for num, file_id in enumerate(per_id_bad_parents):
433
 
            self.pb.update('Fixing text parents', num,
434
 
                           len(per_id_bad_parents))
435
 
            versions_with_bad_parents = per_id_bad_parents[file_id]
436
 
            id_unused_versions = set(key[-1] for key in unused_versions
437
 
                if key[0] == file_id)
438
 
            if file_id in file_id_versions:
439
 
                file_versions = file_id_versions[file_id]
440
 
            else:
441
 
                # This id was present in the disk store but is not referenced
442
 
                # by any revision at all.
443
 
                file_versions = []
444
 
            self._fix_text_parent(file_id, versions_with_bad_parents,
445
 
                 id_unused_versions, file_versions)
446
 
 
447
 
    def _fix_text_parent(self, file_id, versions_with_bad_parents,
448
 
            unused_versions, all_versions):
449
 
        """Fix bad versionedfile entries in a single versioned file."""
450
 
        mutter('fixing text parent: %r (%d versions)', file_id,
451
 
                len(versions_with_bad_parents))
452
 
        mutter('(%d are unused)', len(unused_versions))
453
 
        new_file_id = 'temp:%s' % file_id
454
 
        new_parents = {}
455
 
        needed_keys = set()
456
 
        for version in all_versions:
457
 
            if version in unused_versions:
458
 
                continue
459
 
            elif version in versions_with_bad_parents:
460
 
                parents = versions_with_bad_parents[version][1]
461
 
            else:
462
 
                pmap = self.repo.texts.get_parent_map([(file_id, version)])
463
 
                parents = [key[-1] for key in pmap[(file_id, version)]]
464
 
            new_parents[(new_file_id, version)] = [
465
 
                (new_file_id, parent) for parent in parents]
466
 
            needed_keys.add((file_id, version))
467
 
        def fix_parents(stream):
468
 
            for record in stream:
469
 
                bytes = record.get_bytes_as('fulltext')
470
 
                new_key = (new_file_id, record.key[-1])
471
 
                parents = new_parents[new_key]
472
 
                yield FulltextContentFactory(new_key, parents, record.sha1, bytes)
473
 
        stream = self.repo.texts.get_record_stream(needed_keys, 'topological', True)
474
 
        self.repo._remove_file_id(new_file_id)
475
 
        self.repo.texts.insert_record_stream(fix_parents(stream))
476
 
        self.repo._remove_file_id(file_id)
477
 
        if len(new_parents):
478
 
            self.repo._move_file_id(new_file_id, file_id)
479
 
 
480
 
 
481
 
class PackReconciler(RepoReconciler):
482
 
    """Reconciler that reconciles a pack based repository.
483
 
 
484
 
    Garbage inventories do not affect ancestry queries, and removal is
485
 
    considerably more expensive as there is no separate versioned file for
486
 
    them, so they are not cleaned. In short it is currently a no-op.
487
 
 
488
 
    In future this may be a good place to hook in annotation cache checking,
489
 
    index recreation etc.
490
 
    """
491
 
 
492
 
    # XXX: The index corruption that _fix_text_parents performs is needed for
493
 
    # packs, but not yet implemented. The basic approach is to:
494
 
    #  - lock the names list
495
 
    #  - perform a customised pack() that regenerates data as needed
496
 
    #  - unlock the names list
497
 
    # https://bugs.edge.launchpad.net/bzr/+bug/154173
498
 
 
499
 
    def _reconcile_steps(self):
500
 
        """Perform the steps to reconcile this repository."""
501
 
        if not self.thorough:
502
 
            return
503
 
        collection = self.repo._pack_collection
504
 
        collection.ensure_loaded()
505
 
        collection.lock_names()
506
 
        try:
507
 
            packs = collection.all_packs()
508
 
            all_revisions = self.repo.all_revision_ids()
509
 
            total_inventories = len(list(
510
 
                collection.inventory_index.combined_index.iter_all_entries()))
511
 
            if len(all_revisions):
512
 
                new_pack =  self.repo._reconcile_pack(collection, packs,
513
 
                    ".reconcile", all_revisions, self.pb)
514
 
                if new_pack is not None:
515
 
                    self._discard_and_save(packs)
516
 
            else:
517
 
                # only make a new pack when there is data to copy.
518
 
                self._discard_and_save(packs)
519
 
            self.garbage_inventories = total_inventories - len(list(
520
 
                collection.inventory_index.combined_index.iter_all_entries()))
521
 
        finally:
522
 
            collection._unlock_names()
523
 
 
524
 
    def _discard_and_save(self, packs):
525
 
        """Discard some packs from the repository.
526
 
 
527
 
        This removes them from the memory index, saves the in-memory index
528
 
        which makes the newly reconciled pack visible and hides the packs to be
529
 
        discarded, and finally renames the packs being discarded into the
530
 
        obsolete packs directory.
531
 
 
532
 
        :param packs: The packs to discard.
533
 
        """
534
 
        for pack in packs:
535
 
            self.repo._pack_collection._remove_pack_from_memory(pack)
536
 
        self.repo._pack_collection._save_pack_names()
537
 
        self.repo._pack_collection._obsolete_packs(packs)
 
328
    def _check_garbage_inventories(self):
 
329
        """Check for garbage inventories which we cannot trust
 
330
 
 
331
        We cant trust them because their pre-requisite file data may not
 
332
        be present - all we know is that their revision was not installed.
 
333
        """
 
334
        inventories = set(self.inventory.versions())
 
335
        revisions = set(self.revisions.versions())
 
336
        garbage = inventories.difference(revisions)
 
337
        self.garbage_inventories = len(garbage)
 
338
        for revision_id in garbage:
 
339
            mutter('Garbage inventory {%s} found.', revision_id)