← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/bundle/serializer/v4.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-05-21 13:36:51 UTC
  • mfrom: (5243.2.1 readdir_cleanup)
  • Revision ID: pqm@pqm.ubuntu.com-20100521133651-p62dndo2giy5ls21
(lifeless) Some cleanups to the readdir pyrex code for a little efficiency
 and to avoid compile warnings. (John A Meinel)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/bundle/serializer/v4.py
1
 
# Copyright (C) 2007-2010 Canonical Ltd
 
1
# Copyright (C) 2007 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
30
30
    serializer,
31
31
    trace,
32
32
    ui,
33
 
    versionedfile as _mod_versionedfile,
34
33
    )
35
34
from bzrlib.bundle import bundle_data, serializer as bundle_serializer
36
 
from bzrlib.i18n import gettext, ngettext
37
35
from bzrlib import bencode
38
36
 
39
37
 
40
 
class _MPDiffInventoryGenerator(_mod_versionedfile._MPDiffGenerator):
41
 
    """Generate Inventory diffs serialized inventories."""
42
 
 
43
 
    def __init__(self, repo, inventory_keys):
44
 
        super(_MPDiffInventoryGenerator, self).__init__(repo.inventories,
45
 
            inventory_keys)
46
 
        self.repo = repo
47
 
        self.sha1s = {}
48
 
 
49
 
    def iter_diffs(self):
50
 
        """Compute the diffs one at a time."""
51
 
        # This is instead of compute_diffs() since we guarantee our ordering of
52
 
        # inventories, we don't have to do any buffering
53
 
        self._find_needed_keys()
54
 
        # We actually use a slightly different ordering. We grab all of the
55
 
        # parents first, and then grab the ordered requests.
56
 
        needed_ids = [k[-1] for k in self.present_parents]
57
 
        needed_ids.extend([k[-1] for k in self.ordered_keys])
58
 
        inv_to_str = self.repo._serializer.write_inventory_to_string
59
 
        for inv in self.repo.iter_inventories(needed_ids):
60
 
            revision_id = inv.revision_id
61
 
            key = (revision_id,)
62
 
            if key in self.present_parents:
63
 
                # Not a key we will transmit, which is a shame, since because
64
 
                # of that bundles don't work with stacked branches
65
 
                parent_ids = None
66
 
            else:
67
 
                parent_ids = [k[-1] for k in self.parent_map[key]]
68
 
            as_bytes = inv_to_str(inv)
69
 
            self._process_one_record(key, (as_bytes,))
70
 
            if parent_ids is None:
71
 
                continue
72
 
            diff = self.diffs.pop(key)
73
 
            sha1 = osutils.sha_string(as_bytes)
74
 
            yield revision_id, parent_ids, sha1, diff
75
 
 
76
 
 
77
38
class BundleWriter(object):
78
39
    """Writer for bundle-format files.
79
40
 
323
284
 
324
285
    def do_write(self):
325
286
        """Write all data to the bundle"""
326
 
        trace.note(ngettext('Bundling %d revision.', 'Bundling %d revisions.',
327
 
                            len(self.revision_ids)), len(self.revision_ids))
 
287
        trace.note('Bundling %d revision(s).', len(self.revision_ids))
328
288
        self.repository.lock_read()
329
289
        try:
330
290
            self.bundle.begin()
388
348
        the other side.
389
349
        """
390
350
        inventory_key_order = [(r,) for r in revision_order]
391
 
        generator = _MPDiffInventoryGenerator(self.repository,
392
 
                                              inventory_key_order)
393
 
        for revision_id, parent_ids, sha1, diff in generator.iter_diffs():
 
351
        parent_map = self.repository.inventories.get_parent_map(
 
352
                            inventory_key_order)
 
353
        missing_keys = set(inventory_key_order).difference(parent_map)
 
354
        if missing_keys:
 
355
            raise errors.RevisionNotPresent(list(missing_keys)[0],
 
356
                                            self.repository.inventories)
 
357
        inv_to_str = self.repository._serializer.write_inventory_to_string
 
358
        # Make sure that we grab the parent texts first
 
359
        just_parents = set()
 
360
        map(just_parents.update, parent_map.itervalues())
 
361
        just_parents.difference_update(parent_map)
 
362
        # Ignore ghost parents
 
363
        present_parents = self.repository.inventories.get_parent_map(
 
364
                            just_parents)
 
365
        ghost_keys = just_parents.difference(present_parents)
 
366
        needed_inventories = list(present_parents) + inventory_key_order
 
367
        needed_inventories = [k[-1] for k in needed_inventories]
 
368
        all_lines = {}
 
369
        for inv in self.repository.iter_inventories(needed_inventories):
 
370
            revision_id = inv.revision_id
 
371
            key = (revision_id,)
 
372
            as_bytes = inv_to_str(inv)
 
373
            # The sha1 is validated as the xml/textual form, not as the
 
374
            # form-in-the-repository
 
375
            sha1 = osutils.sha_string(as_bytes)
 
376
            as_lines = osutils.split_lines(as_bytes)
 
377
            del as_bytes
 
378
            all_lines[key] = as_lines
 
379
            if key in just_parents:
 
380
                # We don't transmit those entries
 
381
                continue
 
382
            # Create an mpdiff for this text, and add it to the output
 
383
            parent_keys = parent_map[key]
 
384
            # See the comment in VF.make_mpdiffs about how this effects
 
385
            # ordering when there are ghosts present. I think we have a latent
 
386
            # bug
 
387
            parent_lines = [all_lines[p_key] for p_key in parent_keys
 
388
                            if p_key not in ghost_keys]
 
389
            diff = multiparent.MultiParent.from_lines(
 
390
                as_lines, parent_lines)
394
391
            text = ''.join(diff.to_patch())
 
392
            parent_ids = [k[-1] for k in parent_keys]
395
393
            self.bundle.add_multiparent_record(text, sha1, parent_ids,
396
394
                                               'inventory', revision_id, None)
397
395