/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/reconcile.py

  • Committer: Kent Gibson
  • Date: 2007-11-12 14:03:35 UTC
  • mto: (2996.1.1 ianc-integration)
  • mto: This revision was merged to the branch mainline in revision 2997.
  • Revision ID: warthog618@gmail.com-20071112140335-d4g28jx86jlz8uzx
When comparing logs using assertEqualDiff, pass the igenerated log first then the expected log so the output reflects what is additional in or missing from the generated log.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
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
 
27
27
 
28
28
 
29
29
from bzrlib import (
30
 
    cleanup,
31
30
    errors,
32
31
    ui,
 
32
    repository,
 
33
    repofmt,
33
34
    )
34
 
from bzrlib.trace import mutter
35
 
from bzrlib.tsort import topo_sort
36
 
from bzrlib.versionedfile import AdapterFactory, FulltextContentFactory
 
35
from bzrlib.trace import mutter, note
 
36
from bzrlib.tsort import TopoSorter
37
37
 
38
38
 
39
39
def reconcile(dir, other=None):
61
61
 
62
62
    def reconcile(self):
63
63
        """Perform reconciliation.
64
 
 
 
64
        
65
65
        After reconciliation the following attributes document found issues:
66
66
        inconsistent_parents: The number of revisions in the repository whose
67
67
                              ancestry was being reported incorrectly.
68
68
        garbage_inventories: The number of inventory objects without revisions
69
69
                             that were garbage collected.
70
 
        fixed_branch_history: None if there was no branch, False if the branch
71
 
                              history was correct, True if the branch history
72
 
                              needed to be re-normalized.
73
70
        """
74
71
        self.pb = ui.ui_factory.nested_progress_bar()
75
72
        try:
79
76
 
80
77
    def _reconcile(self):
81
78
        """Helper function for performing reconciliation."""
82
 
        self._reconcile_branch()
83
 
        self._reconcile_repository()
84
 
 
85
 
    def _reconcile_branch(self):
86
 
        try:
87
 
            self.branch = self.bzrdir.open_branch()
88
 
        except errors.NotBranchError:
89
 
            # Nothing to check here
90
 
            self.fixed_branch_history = None
91
 
            return
92
 
        ui.ui_factory.note('Reconciling branch %s' % self.branch.base)
93
 
        branch_reconciler = self.branch.reconcile(thorough=True)
94
 
        self.fixed_branch_history = branch_reconciler.fixed_history
95
 
 
96
 
    def _reconcile_repository(self):
97
79
        self.repo = self.bzrdir.find_repository()
98
 
        ui.ui_factory.note('Reconciling repository %s' %
99
 
            self.repo.user_url)
 
80
        self.pb.note('Reconciling repository %s',
 
81
                     self.repo.bzrdir.root_transport.base)
100
82
        self.pb.update("Reconciling repository", 0, 1)
101
83
        repo_reconciler = self.repo.reconcile(thorough=True)
102
84
        self.inconsistent_parents = repo_reconciler.inconsistent_parents
103
85
        self.garbage_inventories = repo_reconciler.garbage_inventories
104
86
        if repo_reconciler.aborted:
105
 
            ui.ui_factory.note(
 
87
            self.pb.note(
106
88
                'Reconcile aborted: revision index has inconsistent parents.')
107
 
            ui.ui_factory.note(
 
89
            self.pb.note(
108
90
                'Run "bzr check" for more details.')
109
91
        else:
110
 
            ui.ui_factory.note('Reconciliation complete.')
111
 
 
112
 
 
113
 
class BranchReconciler(object):
114
 
    """Reconciler that works on a branch."""
115
 
 
116
 
    def __init__(self, a_branch, thorough=False):
117
 
        self.fixed_history = None
118
 
        self.thorough = thorough
119
 
        self.branch = a_branch
120
 
 
121
 
    def reconcile(self):
122
 
        operation = cleanup.OperationWithCleanups(self._reconcile)
123
 
        self.add_cleanup = operation.add_cleanup
124
 
        operation.run_simple()
125
 
 
126
 
    def _reconcile(self):
127
 
        self.branch.lock_write()
128
 
        self.add_cleanup(self.branch.unlock)
129
 
        self.pb = ui.ui_factory.nested_progress_bar()
130
 
        self.add_cleanup(self.pb.finished)
131
 
        self._reconcile_steps()
132
 
 
133
 
    def _reconcile_steps(self):
134
 
        self._reconcile_revision_history()
135
 
 
136
 
    def _reconcile_revision_history(self):
137
 
        repo = self.branch.repository
138
 
        last_revno, last_revision_id = self.branch.last_revision_info()
139
 
        real_history = []
140
 
        try:
141
 
            for revid in repo.iter_reverse_revision_history(
142
 
                    last_revision_id):
143
 
                real_history.append(revid)
144
 
        except errors.RevisionNotPresent:
145
 
            pass # Hit a ghost left hand parent
146
 
        real_history.reverse()
147
 
        if last_revno != len(real_history):
148
 
            self.fixed_history = True
149
 
            # Technically for Branch5 formats, it is more efficient to use
150
 
            # set_revision_history, as this will regenerate it again.
151
 
            # Not really worth a whole BranchReconciler class just for this,
152
 
            # though.
153
 
            ui.ui_factory.note('Fixing last revision info %s => %s' % (
154
 
                 last_revno, len(real_history)))
155
 
            self.branch.set_last_revision_info(len(real_history),
156
 
                                               last_revision_id)
157
 
        else:
158
 
            self.fixed_history = False
159
 
            ui.ui_factory.note('revision_history ok.')
 
92
            self.pb.note('Reconciliation complete.')
160
93
 
161
94
 
162
95
class RepoReconciler(object):
163
96
    """Reconciler that reconciles a repository.
164
97
 
165
98
    The goal of repository reconciliation is to make any derived data
166
 
    consistent with the core data committed by a user. This can involve
 
99
    consistent with the core data committed by a user. This can involve 
167
100
    reindexing, or removing unreferenced data if that can interfere with
168
101
    queries in a given repository.
169
102
 
185
118
 
186
119
    def reconcile(self):
187
120
        """Perform reconciliation.
188
 
 
 
121
        
189
122
        After reconciliation the following attributes document found issues:
190
123
        inconsistent_parents: The number of revisions in the repository whose
191
124
                              ancestry was being reported incorrectly.
192
125
        garbage_inventories: The number of inventory objects without revisions
193
126
                             that were garbage collected.
194
127
        """
195
 
        operation = cleanup.OperationWithCleanups(self._reconcile)
196
 
        self.add_cleanup = operation.add_cleanup
197
 
        operation.run_simple()
198
 
 
199
 
    def _reconcile(self):
200
128
        self.repo.lock_write()
201
 
        self.add_cleanup(self.repo.unlock)
202
 
        self.pb = ui.ui_factory.nested_progress_bar()
203
 
        self.add_cleanup(self.pb.finished)
204
 
        self._reconcile_steps()
 
129
        try:
 
130
            self.pb = ui.ui_factory.nested_progress_bar()
 
131
            try:
 
132
                self._reconcile_steps()
 
133
            finally:
 
134
                self.pb.finished()
 
135
        finally:
 
136
            self.repo.unlock()
205
137
 
206
138
    def _reconcile_steps(self):
207
139
        """Perform the steps to reconcile this repository."""
209
141
 
210
142
    def _reweave_inventory(self):
211
143
        """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
 
144
        
 
145
        This is a smart function: it will only do the reweave if doing it 
214
146
        will correct data issues. The self.thorough flag controls whether
215
147
        only data-loss causing issues (!self.thorough) or all issues
216
148
        (self.thorough) are treated as requiring the reweave.
218
150
        # local because needing to know about WeaveFile is a wart we want to hide
219
151
        from bzrlib.weave import WeaveFile, Weave
220
152
        transaction = self.repo.get_transaction()
221
 
        self.pb.update('Reading inventory data')
222
 
        self.inventory = self.repo.inventories
223
 
        self.revisions = self.repo.revisions
 
153
        self.pb.update('Reading inventory data.')
 
154
        self.inventory = self.repo.get_inventory_weave()
224
155
        # the total set of revisions to process
225
 
        self.pending = set([key[-1] for key in self.revisions.keys()])
 
156
        self.pending = set([rev_id for rev_id in self.repo._revision_store.all_revision_ids(transaction)])
226
157
 
227
158
        # mapping from revision_id to parents
228
159
        self._rev_graph = {}
234
165
            # put a revision into the graph.
235
166
            self._graph_revision(rev_id)
236
167
        self._check_garbage_inventories()
237
 
        # if there are no inconsistent_parents and
 
168
        # if there are no inconsistent_parents and 
238
169
        # (no garbage inventories or we are not doing a thorough check)
239
 
        if (not self.inconsistent_parents and
 
170
        if (not self.inconsistent_parents and 
240
171
            (not self.garbage_inventories or not self.thorough)):
241
 
            ui.ui_factory.note('Inventory ok.')
 
172
            self.pb.note('Inventory ok.')
242
173
            return
243
 
        self.pb.update('Backing up inventory', 0, 0)
244
 
        self.repo._backup_inventory()
245
 
        ui.ui_factory.note('Backup inventory created.')
246
 
        new_inventories = self.repo._temp_inventories()
 
174
        self.pb.update('Backing up inventory...', 0, 0)
 
175
        self.repo.control_weaves.copy(self.inventory, 'inventory.backup', self.repo.get_transaction())
 
176
        self.pb.note('Backup Inventory created.')
 
177
        # asking for '' should never return a non-empty weave
 
178
        new_inventory_vf = self.repo.control_weaves.get_empty('inventory.new',
 
179
            self.repo.get_transaction())
247
180
 
248
181
        # we have topological order of revisions and non ghost parents ready.
249
182
        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
 
183
        for rev_id in TopoSorter(self._rev_graph.items()).iter_topo_order():
 
184
            parents = self._rev_graph[rev_id]
 
185
            # double check this really is in topological order.
 
186
            unavailable = [p for p in parents if p not in new_inventory_vf]
 
187
            assert len(unavailable) == 0
 
188
            # this entry has all the non ghost parents in the inventory
 
189
            # file already.
 
190
            self._reweave_step('adding inventories')
 
191
            if isinstance(new_inventory_vf, WeaveFile):
 
192
                # It's really a WeaveFile, but we call straight into the
 
193
                # Weave's add method to disable the auto-write-out behaviour.
 
194
                # This is done to avoid a revision_count * time-to-write additional overhead on 
 
195
                # reconcile.
 
196
                new_inventory_vf._check_write_ok()
 
197
                Weave._add_lines(new_inventory_vf, rev_id, parents,
 
198
                    self.inventory.get_lines(rev_id), None, None, None, False, True)
 
199
            else:
 
200
                new_inventory_vf.add_lines(rev_id, parents, self.inventory.get_lines(rev_id))
 
201
 
 
202
        if isinstance(new_inventory_vf, WeaveFile):
 
203
            new_inventory_vf._save()
 
204
        # if this worked, the set of new_inventory_vf.names should equal
257
205
        # self.pending
258
 
        if not (set(new_inventories.keys()) ==
259
 
            set([(revid,) for revid in self.pending])):
260
 
            raise AssertionError()
 
206
        assert set(new_inventory_vf.versions()) == self.pending
261
207
        self.pb.update('Writing weave')
262
 
        self.repo._activate_new_inventory()
 
208
        self.repo.control_weaves.copy(new_inventory_vf, 'inventory', self.repo.get_transaction())
 
209
        self.repo.control_weaves.delete('inventory.new', self.repo.get_transaction())
263
210
        self.inventory = None
264
 
        ui.ui_factory.note('Inventory regenerated.')
265
 
 
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')
 
211
        self.pb.note('Inventory regenerated.')
285
212
 
286
213
    def _setup_steps(self, new_total):
287
214
        """Setup the markers we need to control the progress bar."""
294
221
        # analyse revision id rev_id and put it in the stack.
295
222
        self._reweave_step('loading revisions')
296
223
        rev = self.repo.get_revision_reconcile(rev_id)
 
224
        assert rev.revision_id == rev_id
297
225
        parents = []
298
226
        for parent in rev.parent_ids:
299
227
            if self._parent_is_available(parent):
300
228
                parents.append(parent)
301
229
            else:
302
230
                mutter('found ghost %s', parent)
303
 
        self._rev_graph[rev_id] = parents
 
231
        self._rev_graph[rev_id] = parents   
 
232
        if self._parents_are_inconsistent(rev_id, parents):
 
233
            self.inconsistent_parents += 1
 
234
            mutter('Inconsistent inventory parents: id {%s} '
 
235
                   'inventory claims %r, '
 
236
                   'available parents are %r, '
 
237
                   'unavailable parents are %r',
 
238
                   rev_id, 
 
239
                   set(self.inventory.get_parents(rev_id)),
 
240
                   set(parents),
 
241
                   set(rev.parent_ids).difference(set(parents)))
 
242
 
 
243
    def _parents_are_inconsistent(self, rev_id, parents):
 
244
        """Return True if the parents list of rev_id does not match the weave.
 
245
 
 
246
        This detects inconsistencies based on the self.thorough value:
 
247
        if thorough is on, the first parent value is checked as well as ghost
 
248
        differences.
 
249
        Otherwise only the ghost differences are evaluated.
 
250
        """
 
251
        weave_parents = self.inventory.get_parents(rev_id)
 
252
        weave_missing_old_ghosts = set(weave_parents) != set(parents)
 
253
        first_parent_is_wrong = (
 
254
            len(weave_parents) and len(parents) and
 
255
            parents[0] != weave_parents[0])
 
256
        if self.thorough:
 
257
            return weave_missing_old_ghosts or first_parent_is_wrong
 
258
        else:
 
259
            return weave_missing_old_ghosts
304
260
 
305
261
    def _check_garbage_inventories(self):
306
262
        """Check for garbage inventories which we cannot trust
310
266
        """
311
267
        if not self.thorough:
312
268
            return
313
 
        inventories = set(self.inventory.keys())
314
 
        revisions = set(self.revisions.keys())
 
269
        inventories = set(self.inventory.versions())
 
270
        revisions = set(self._rev_graph.keys())
315
271
        garbage = inventories.difference(revisions)
316
272
        self.garbage_inventories = len(garbage)
317
 
        for revision_key in garbage:
318
 
            mutter('Garbage inventory {%s} found.', revision_key[-1])
 
273
        for revision_id in garbage:
 
274
            mutter('Garbage inventory {%s} found.', revision_id)
319
275
 
320
276
    def _parent_is_available(self, parent):
321
277
        """True if parent is a fully available revision
323
279
        A fully available revision has a inventory and a revision object in the
324
280
        repository.
325
281
        """
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))
 
282
        return (parent in self._rev_graph or 
 
283
                (parent in self.inventory and self.repo.has_revision(parent)))
330
284
 
331
285
    def _reweave_step(self, message):
332
286
        """Mark a single step of regeneration complete."""
355
309
    def _load_indexes(self):
356
310
        """Load indexes for the reconciliation."""
357
311
        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)
 
312
        self.pb.update('Reading indexes.', 0, 2)
 
313
        self.inventory = self.repo.get_inventory_weave()
 
314
        self.pb.update('Reading indexes.', 1, 2)
361
315
        self.repo._check_for_inconsistent_revision_parents()
362
 
        self.revisions = self.repo.revisions
363
 
        self.pb.update('Reading indexes', 2, 2)
 
316
        self.revisions = self.repo._revision_store.get_revision_file(self.transaction)
 
317
        self.pb.update('Reading indexes.', 2, 2)
364
318
 
365
319
    def _gc_inventory(self):
366
320
        """Remove inventories that are not referenced from the revision store."""
367
 
        self.pb.update('Checking unused inventories', 0, 1)
 
321
        self.pb.update('Checking unused inventories.', 0, 1)
368
322
        self._check_garbage_inventories()
369
 
        self.pb.update('Checking unused inventories', 1, 3)
 
323
        self.pb.update('Checking unused inventories.', 1, 3)
370
324
        if not self.garbage_inventories:
371
 
            ui.ui_factory.note('Inventory ok.')
 
325
            self.pb.note('Inventory ok.')
372
326
            return
373
 
        self.pb.update('Backing up inventory', 0, 0)
374
 
        self.repo._backup_inventory()
375
 
        ui.ui_factory.note('Backup Inventory created')
 
327
        self.pb.update('Backing up inventory...', 0, 0)
 
328
        self.repo.control_weaves.copy(self.inventory, 'inventory.backup', self.transaction)
 
329
        self.pb.note('Backup Inventory created.')
376
330
        # asking for '' should never return a non-empty weave
377
 
        new_inventories = self.repo._temp_inventories()
 
331
        new_inventory_vf = self.repo.control_weaves.get_empty('inventory.new',
 
332
            self.transaction)
 
333
 
378
334
        # 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)
 
335
        self._setup_steps(len(self.revisions))
 
336
        for rev_id in TopoSorter(self.revisions.get_graph().items()).iter_topo_order():
 
337
            parents = self.revisions.get_parents(rev_id)
 
338
            # double check this really is in topological order.
 
339
            unavailable = [p for p in parents if p not in new_inventory_vf]
 
340
            assert len(unavailable) == 0
 
341
            # this entry has all the non ghost parents in the inventory
 
342
            # file already.
 
343
            self._reweave_step('adding inventories')
 
344
            # ugly but needed, weaves are just way tooooo slow else.
 
345
            new_inventory_vf.add_lines(rev_id, parents, self.inventory.get_lines(rev_id))
 
346
 
388
347
        # 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()
 
348
        # self.pending
 
349
        assert set(new_inventory_vf.versions()) == set(self.revisions.versions())
392
350
        self.pb.update('Writing weave')
393
 
        self.repo._activate_new_inventory()
 
351
        self.repo.control_weaves.copy(new_inventory_vf, 'inventory', self.transaction)
 
352
        self.repo.control_weaves.delete('inventory.new', self.transaction)
394
353
        self.inventory = None
395
 
        ui.ui_factory.note('Inventory regenerated.')
 
354
        self.pb.note('Inventory regenerated.')
 
355
 
 
356
    def _check_garbage_inventories(self):
 
357
        """Check for garbage inventories which we cannot trust
 
358
 
 
359
        We cant trust them because their pre-requisite file data may not
 
360
        be present - all we know is that their revision was not installed.
 
361
        """
 
362
        inventories = set(self.inventory.versions())
 
363
        revisions = set(self.revisions.versions())
 
364
        garbage = inventories.difference(revisions)
 
365
        self.garbage_inventories = len(garbage)
 
366
        for revision_id in garbage:
 
367
            mutter('Garbage inventory {%s} found.', revision_id)
396
368
 
397
369
    def _fix_text_parents(self):
398
370
        """Fix bad versionedfile parent entries.
404
376
        parent lists, and replaces the versionedfile with a corrected version.
405
377
        """
406
378
        transaction = self.repo.get_transaction()
407
 
        versions = [key[-1] for key in self.revisions.keys()]
 
379
        revision_versions = repository._RevisionTextVersionCache(self.repo)
 
380
        versions = self.revisions.versions()
408
381
        mutter('Prepopulating revision text cache with %d revisions',
409
382
                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):
 
383
        revision_versions.prepopulate_revs(versions)
 
384
        used_file_versions = revision_versions.used_file_versions()
 
385
        for num, file_id in enumerate(self.repo.weave_store):
433
386
            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)
 
387
                           len(self.repo.weave_store))
 
388
            vf = self.repo.weave_store.get_weave(file_id, transaction)
 
389
            vf_checker = self.repo.get_versioned_file_checker(
 
390
                vf.versions(), revision_versions)
 
391
            versions_with_bad_parents, dangling_file_versions = \
 
392
                vf_checker.check_file_version_parents(vf, file_id)
 
393
            if (len(versions_with_bad_parents) == 0 and
 
394
                len(dangling_file_versions) == 0):
 
395
                continue
 
396
            full_text_versions = set()
 
397
            unused_versions = set()
 
398
            for dangling_version in dangling_file_versions:
 
399
                version = dangling_version[1]
 
400
                if dangling_version in used_file_versions:
 
401
                    # This version *is* used by some revision, even though it
 
402
                    # isn't used by its own revision!  We make sure any
 
403
                    # revision referencing it is stored as a fulltext
 
404
                    # This avoids bug 155730: it means that clients looking at
 
405
                    # inventories to determine the versions to fetch will not
 
406
                    # miss a required version.  (So clients can assume that if
 
407
                    # they have a complete revision graph, and fetch all file
 
408
                    # versions named by those revisions inventories, then they
 
409
                    # will not have any missing parents for 'delta' knit
 
410
                    # records.)
 
411
                    # XXX: A better, but more difficult and slower fix would be
 
412
                    # to rewrite the inventories referencing this version.
 
413
                    full_text_versions.add(version)
 
414
                else:
 
415
                    # This version is totally unreferenced.  It should be
 
416
                    # removed.
 
417
                    unused_versions.add(version)
 
418
            self._fix_text_parent(file_id, vf, versions_with_bad_parents,
 
419
                full_text_versions, unused_versions)
446
420
 
447
 
    def _fix_text_parent(self, file_id, versions_with_bad_parents,
448
 
            unused_versions, all_versions):
 
421
    def _fix_text_parent(self, file_id, vf, versions_with_bad_parents,
 
422
            full_text_versions, unused_versions):
449
423
        """Fix bad versionedfile entries in a single versioned file."""
450
424
        mutter('fixing text parent: %r (%d versions)', file_id,
451
425
                len(versions_with_bad_parents))
452
 
        mutter('(%d are unused)', len(unused_versions))
453
 
        new_file_id = 'temp:%s' % file_id
 
426
        mutter('(%d need to be full texts, %d are unused)',
 
427
                len(full_text_versions), len(unused_versions))
 
428
        new_vf = self.repo.weave_store.get_empty('temp:%s' % file_id,
 
429
            self.transaction)
454
430
        new_parents = {}
455
 
        needed_keys = set()
456
 
        for version in all_versions:
 
431
        for version in vf.versions():
 
432
            if version in versions_with_bad_parents:
 
433
                parents = versions_with_bad_parents[version][1]
 
434
            else:
 
435
                parents = vf.get_parents(version)
 
436
            new_parents[version] = parents
 
437
        for version in TopoSorter(new_parents.items()).iter_topo_order():
457
438
            if version in unused_versions:
458
439
                continue
459
 
            elif version in versions_with_bad_parents:
460
 
                parents = versions_with_bad_parents[version][1]
 
440
            lines = vf.get_lines(version)
 
441
            parents = new_parents[version]
 
442
            if parents and (parents[0] in full_text_versions):
 
443
                # Force this record to be a fulltext, not a delta.
 
444
                new_vf._add(version, lines, parents, False,
 
445
                    None, None, None, False)
461
446
            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)
 
447
                new_vf.add_lines(version, parents, lines)
 
448
        self.repo.weave_store.copy(new_vf, file_id, self.transaction)
 
449
        self.repo.weave_store.delete('temp:%s' % file_id, self.transaction)
479
450
 
480
451
 
481
452
class PackReconciler(RepoReconciler):
503
474
        collection = self.repo._pack_collection
504
475
        collection.ensure_loaded()
505
476
        collection.lock_names()
506
 
        self.add_cleanup(collection._unlock_names)
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:
 
477
        try:
 
478
            packs = collection.all_packs()
 
479
            all_revisions = self.repo.all_revision_ids()
 
480
            total_inventories = len(list(
 
481
                collection.inventory_index.combined_index.iter_all_entries()))
 
482
            if len(all_revisions):
 
483
                self._packer = repofmt.pack_repo.ReconcilePacker(
 
484
                    collection, packs, ".reconcile", all_revisions)
 
485
                new_pack = self._packer.pack(pb=self.pb)
 
486
                if new_pack is not None:
 
487
                    self._discard_and_save(packs)
 
488
            else:
 
489
                # only make a new pack when there is data to copy.
515
490
                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()))
 
491
            self.garbage_inventories = total_inventories - len(list(
 
492
                collection.inventory_index.combined_index.iter_all_entries()))
 
493
        finally:
 
494
            collection._unlock_names()
521
495
 
522
496
    def _discard_and_save(self, packs):
523
497
        """Discard some packs from the repository.