← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/repository.py

  • Committer: Vincent Ladeuil
  • Date: 2011-01-20 21:15:10 UTC
  • mfrom: (5050.62.4 2.2)
  • mto: (5609.2.4 2.3)
  • mto: This revision was merged to the branch mainline in revision 5628.
  • Revision ID: v.ladeuil+lp@free.fr-20110120211510-9dl4tbl77dad86pl
Merge 2.2 into 2.3 including bugfix for #701940

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/repository.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005-2011 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
25
25
    check,
26
26
    chk_map,
27
27
    config,
 
28
    controldir,
28
29
    debug,
29
 
    errors,
30
30
    fetch as _mod_fetch,
31
31
    fifo_cache,
32
32
    generate_ids,
39
39
    lockdir,
40
40
    lru_cache,
41
41
    osutils,
 
42
    pyutils,
42
43
    revision as _mod_revision,
43
44
    static_tuple,
44
45
    symbol_versioning,
45
46
    trace,
46
47
    tsort,
47
 
    ui,
48
48
    versionedfile,
49
49
    )
50
50
from bzrlib.bundle import serializer
53
53
from bzrlib.testament import Testament
54
54
""")
55
55
 
 
56
import sys
 
57
from bzrlib import (
 
58
    errors,
 
59
    registry,
 
60
    ui,
 
61
    )
56
62
from bzrlib.decorators import needs_read_lock, needs_write_lock, only_raises
57
63
from bzrlib.inter import InterObject
58
64
from bzrlib.inventory import (
61
67
    ROOT_ID,
62
68
    entry_factory,
63
69
    )
64
 
from bzrlib.lock import _RelockDebugMixin
65
 
from bzrlib import registry
 
70
from bzrlib.recordcounter import RecordCounter
 
71
from bzrlib.lock import _RelockDebugMixin, LogicalLockResult
66
72
from bzrlib.trace import (
67
73
    log_exception_quietly, note, mutter, mutter_callsite, warning)
68
74
 
71
77
_deprecation_warning_done = False
72
78
 
73
79
 
 
80
class IsInWriteGroupError(errors.InternalBzrError):
 
81
 
 
82
    _fmt = "May not refresh_data of repo %(repo)s while in a write group."
 
83
 
 
84
    def __init__(self, repo):
 
85
        errors.InternalBzrError.__init__(self, repo=repo)
 
86
 
 
87
 
74
88
class CommitBuilder(object):
75
89
    """Provides an interface to build up a commit.
76
90
 
101
115
 
102
116
        if committer is None:
103
117
            self._committer = self._config.username()
 
118
        elif not isinstance(committer, unicode):
 
119
            self._committer = committer.decode() # throw if non-ascii
104
120
        else:
105
121
            self._committer = committer
106
122
 
160
176
            self._validate_unicode_text(value,
161
177
                                        'revision property (%s)' % (key,))
162
178
 
 
179
    def _ensure_fallback_inventories(self):
 
180
        """Ensure that appropriate inventories are available.
 
181
 
 
182
        This only applies to repositories that are stacked, and is about
 
183
        enusring the stacking invariants. Namely, that for any revision that is
 
184
        present, we either have all of the file content, or we have the parent
 
185
        inventory and the delta file content.
 
186
        """
 
187
        if not self.repository._fallback_repositories:
 
188
            return
 
189
        if not self.repository._format.supports_chks:
 
190
            raise errors.BzrError("Cannot commit directly to a stacked branch"
 
191
                " in pre-2a formats. See "
 
192
                "https://bugs.launchpad.net/bzr/+bug/375013 for details.")
 
193
        # This is a stacked repo, we need to make sure we have the parent
 
194
        # inventories for the parents.
 
195
        parent_keys = [(p,) for p in self.parents]
 
196
        parent_map = self.repository.inventories._index.get_parent_map(parent_keys)
 
197
        missing_parent_keys = set([pk for pk in parent_keys
 
198
                                       if pk not in parent_map])
 
199
        fallback_repos = list(reversed(self.repository._fallback_repositories))
 
200
        missing_keys = [('inventories', pk[0])
 
201
                        for pk in missing_parent_keys]
 
202
        resume_tokens = []
 
203
        while missing_keys and fallback_repos:
 
204
            fallback_repo = fallback_repos.pop()
 
205
            source = fallback_repo._get_source(self.repository._format)
 
206
            sink = self.repository._get_sink()
 
207
            stream = source.get_stream_for_missing_keys(missing_keys)
 
208
            missing_keys = sink.insert_stream_without_locking(stream,
 
209
                self.repository._format)
 
210
        if missing_keys:
 
211
            raise errors.BzrError('Unable to fill in parent inventories for a'
 
212
                                  ' stacked branch')
 
213
 
163
214
    def commit(self, message):
164
215
        """Make the actual commit.
165
216
 
177
228
        rev.parent_ids = self.parents
178
229
        self.repository.add_revision(self._new_revision_id, rev,
179
230
            self.new_inventory, self._config)
 
231
        self._ensure_fallback_inventories()
180
232
        self.repository.commit_write_group()
181
233
        return self._new_revision_id
182
234
 
231
283
 
232
284
    def _gen_revision_id(self):
233
285
        """Return new revision-id."""
234
 
        return generate_ids.gen_revision_id(self._config.username(),
235
 
                                            self._timestamp)
 
286
        return generate_ids.gen_revision_id(self._committer, self._timestamp)
236
287
 
237
288
    def _generate_revision_if_needed(self):
238
289
        """Create a revision id if None was supplied.
278
329
 
279
330
        :param tree: The tree which is being committed.
280
331
        """
281
 
        # NB: if there are no parents then this method is not called, so no
282
 
        # need to guard on parents having length.
 
332
        if len(self.parents) == 0:
 
333
            raise errors.RootMissing()
283
334
        entry = entry_factory['directory'](tree.path2id(''), '',
284
335
            None)
285
336
        entry.revision = self._new_revision_id
423
474
            else:
424
475
                # we don't need to commit this, because the caller already
425
476
                # determined that an existing revision of this file is
426
 
                # appropriate. If its not being considered for committing then
 
477
                # appropriate. If it's not being considered for committing then
427
478
                # it and all its parents to the root must be unaltered so
428
479
                # no-change against the basis.
429
480
                if ie.revision == self._new_revision_id:
745
796
                    # after iter_changes examines and decides it has changed,
746
797
                    # we will unconditionally record a new version even if some
747
798
                    # other process reverts it while commit is running (with
748
 
                    # the revert happening after iter_changes did it's
 
799
                    # the revert happening after iter_changes did its
749
800
                    # examination).
750
801
                    if change[7][1]:
751
802
                        entry.executable = True
860
911
        # versioned roots do not change unless the tree found a change.
861
912
 
862
913
 
863
 
class RepositoryWriteLockResult(object):
 
914
class RepositoryWriteLockResult(LogicalLockResult):
864
915
    """The result of write locking a repository.
865
916
 
866
917
    :ivar repository_token: The token obtained from the underlying lock, or
869
920
    """
870
921
 
871
922
    def __init__(self, unlock, repository_token):
 
923
        LogicalLockResult.__init__(self, unlock)
872
924
        self.repository_token = repository_token
873
 
        self.unlock = unlock
874
925
 
875
 
    def __str__(self):
 
926
    def __repr__(self):
876
927
        return "RepositoryWriteLockResult(%s, %s)" % (self.repository_token,
877
928
            self.unlock)
878
929
 
881
932
# Repositories
882
933
 
883
934
 
884
 
class Repository(_RelockDebugMixin, bzrdir.ControlComponent):
 
935
class Repository(_RelockDebugMixin, controldir.ControlComponent):
885
936
    """Repository holding history for one or more branches.
886
937
 
887
938
    The repository holds and retrieves historical information including
934
985
        pointing to .bzr/repository.
935
986
    """
936
987
 
937
 
    # What class to use for a CommitBuilder. Often its simpler to change this
 
988
    # What class to use for a CommitBuilder. Often it's simpler to change this
938
989
    # in a Repository class subclass rather than to override
939
990
    # get_commit_builder.
940
991
    _commit_builder_class = CommitBuilder
1035
1086
                " id and insertion revid (%r, %r)"
1036
1087
                % (inv.revision_id, revision_id))
1037
1088
        if inv.root is None:
1038
 
            raise AssertionError()
 
1089
            raise errors.RootMissing()
1039
1090
        return self._add_inventory_checked(revision_id, inv, parents)
1040
1091
 
1041
1092
    def _add_inventory_checked(self, revision_id, inv, parents):
1434
1485
            for repo in self._fallback_repositories:
1435
1486
                repo.lock_read()
1436
1487
            self._refresh_data()
1437
 
        return self
 
1488
        return LogicalLockResult(self.unlock)
1438
1489
 
1439
1490
    def get_physical_lock_status(self):
1440
1491
        return self.control_files.get_physical_lock_status()
1658
1709
        return missing_keys
1659
1710
 
1660
1711
    def refresh_data(self):
1661
 
        """Re-read any data needed to to synchronise with disk.
 
1712
        """Re-read any data needed to synchronise with disk.
1662
1713
 
1663
1714
        This method is intended to be called after another repository instance
1664
1715
        (such as one used by a smart server) has inserted data into the
1665
 
        repository. It may not be called during a write group, but may be
1666
 
        called at any other time.
 
1716
        repository. On all repositories this will work outside of write groups.
 
1717
        Some repository formats (pack and newer for bzrlib native formats)
 
1718
        support refresh_data inside write groups. If called inside a write
 
1719
        group on a repository that does not support refreshing in a write group
 
1720
        IsInWriteGroupError will be raised.
1667
1721
        """
1668
 
        if self.is_in_write_group():
1669
 
            raise errors.InternalBzrError(
1670
 
                "May not refresh_data while in a write group.")
1671
1722
        self._refresh_data()
1672
1723
 
1673
1724
    def resume_write_group(self, tokens):
1712
1763
                "May not fetch while in a write group.")
1713
1764
        # fast path same-url fetch operations
1714
1765
        # TODO: lift out to somewhere common with RemoteRepository
1715
 
        # <https://bugs.edge.launchpad.net/bzr/+bug/401646>
 
1766
        # <https://bugs.launchpad.net/bzr/+bug/401646>
1716
1767
        if (self.has_same_location(source)
1717
1768
            and fetch_spec is None
1718
1769
            and self._has_same_fallbacks(source)):
1746
1797
        :param revprops: Optional dictionary of revision properties.
1747
1798
        :param revision_id: Optional revision id.
1748
1799
        """
1749
 
        if self._fallback_repositories:
1750
 
            raise errors.BzrError("Cannot commit from a lightweight checkout "
1751
 
                "to a stacked branch. See "
 
1800
        if self._fallback_repositories and not self._format.supports_chks:
 
1801
            raise errors.BzrError("Cannot commit directly to a stacked branch"
 
1802
                " in pre-2a formats. See "
1752
1803
                "https://bugs.launchpad.net/bzr/+bug/375013 for details.")
1753
1804
        result = self._commit_builder_class(self, parents, config,
1754
1805
            timestamp, timezone, committer, revprops, revision_id)
2500
2551
            ancestors will be traversed.
2501
2552
        """
2502
2553
        graph = self.get_graph()
2503
 
        next_id = revision_id
2504
 
        while True:
2505
 
            if next_id in (None, _mod_revision.NULL_REVISION):
2506
 
                return
2507
 
            try:
2508
 
                parents = graph.get_parent_map([next_id])[next_id]
2509
 
            except KeyError:
2510
 
                raise errors.RevisionNotPresent(next_id, self)
2511
 
            yield next_id
2512
 
            if len(parents) == 0:
2513
 
                return
2514
 
            else:
2515
 
                next_id = parents[0]
 
2554
        stop_revisions = (None, _mod_revision.NULL_REVISION)
 
2555
        return graph.iter_lefthand_ancestry(revision_id, stop_revisions)
2516
2556
 
2517
2557
    def is_shared(self):
2518
2558
        """Return True if this repository is flagged as a shared repository."""
2619
2659
        types it should be a no-op that just returns.
2620
2660
 
2621
2661
        This stub method does not require a lock, but subclasses should use
2622
 
        @needs_write_lock as this is a long running call its reasonable to
 
2662
        @needs_write_lock as this is a long running call it's reasonable to
2623
2663
        implicitly lock for the user.
2624
2664
 
2625
2665
        :param hint: If not supplied, the whole repository is packed.
2818
2858
        raise NotImplementedError(self.revision_graph_can_have_wrong_parents)
2819
2859
 
2820
2860
 
2821
 
# remove these delegates a while after bzr 0.15
2822
 
def __make_delegated(name, from_module):
2823
 
    def _deprecated_repository_forwarder():
2824
 
        symbol_versioning.warn('%s moved to %s in bzr 0.15'
2825
 
            % (name, from_module),
2826
 
            DeprecationWarning,
2827
 
            stacklevel=2)
2828
 
        m = __import__(from_module, globals(), locals(), [name])
2829
 
        try:
2830
 
            return getattr(m, name)
2831
 
        except AttributeError:
2832
 
            raise AttributeError('module %s has no name %s'
2833
 
                    % (m, name))
2834
 
    globals()[name] = _deprecated_repository_forwarder
2835
 
 
2836
 
for _name in [
2837
 
        'AllInOneRepository',
2838
 
        'WeaveMetaDirRepository',
2839
 
        'PreSplitOutRepositoryFormat',
2840
 
        'RepositoryFormat4',
2841
 
        'RepositoryFormat5',
2842
 
        'RepositoryFormat6',
2843
 
        'RepositoryFormat7',
2844
 
        ]:
2845
 
    __make_delegated(_name, 'bzrlib.repofmt.weaverepo')
2846
 
 
2847
 
for _name in [
2848
 
        'KnitRepository',
2849
 
        'RepositoryFormatKnit',
2850
 
        'RepositoryFormatKnit1',
2851
 
        ]:
2852
 
    __make_delegated(_name, 'bzrlib.repofmt.knitrepo')
2853
 
 
2854
 
 
2855
2861
def install_revision(repository, rev, revision_tree):
2856
2862
    """Install all revision data into a repository."""
2857
2863
    install_revisions(repository, [(rev, revision_tree, None)])
3076
3082
    supports_tree_reference = None
3077
3083
    # Is the format experimental ?
3078
3084
    experimental = False
 
3085
    # Does this repository format escape funky characters, or does it create files with
 
3086
    # similar names as the versioned files in its contents on disk ?
 
3087
    supports_funky_characters = True
3079
3088
 
3080
3089
    def __repr__(self):
3081
3090
        return "%s()" % self.__class__.__name__
3349
3358
    'bzrlib.repofmt.pack_repo',
3350
3359
    'RepositoryFormatKnitPack6RichRoot',
3351
3360
    )
 
3361
format_registry.register_lazy(
 
3362
    'Bazaar repository format 2a (needs bzr 1.16 or later)\n',
 
3363
    'bzrlib.repofmt.groupcompress_repo',
 
3364
    'RepositoryFormat2a',
 
3365
    )
3352
3366
 
3353
3367
# Development formats.
3354
 
# Obsolete but kept pending a CHK based subtree format.
 
3368
# Check their docstrings to see if/when they are obsolete.
3355
3369
format_registry.register_lazy(
3356
3370
    ("Bazaar development format 2 with subtree support "
3357
3371
        "(needs bzr.dev from before 1.8)\n"),
3358
3372
    'bzrlib.repofmt.pack_repo',
3359
3373
    'RepositoryFormatPackDevelopment2Subtree',
3360
3374
    )
3361
 
 
3362
 
# 1.14->1.16 go below here
3363
 
format_registry.register_lazy(
3364
 
    'Bazaar development format - group compression and chk inventory'
3365
 
        ' (needs bzr.dev from 1.14)\n',
3366
 
    'bzrlib.repofmt.groupcompress_repo',
3367
 
    'RepositoryFormatCHK1',
3368
 
    )
3369
 
 
3370
 
format_registry.register_lazy(
3371
 
    'Bazaar development format - chk repository with bencode revision '
3372
 
        'serialization (needs bzr.dev from 1.16)\n',
3373
 
    'bzrlib.repofmt.groupcompress_repo',
3374
 
    'RepositoryFormatCHK2',
3375
 
    )
3376
 
format_registry.register_lazy(
3377
 
    'Bazaar repository format 2a (needs bzr 1.16 or later)\n',
3378
 
    'bzrlib.repofmt.groupcompress_repo',
3379
 
    'RepositoryFormat2a',
 
3375
format_registry.register_lazy(
 
3376
    'Bazaar development format 8\n',
 
3377
    'bzrlib.repofmt.groupcompress_repo',
 
3378
    'RepositoryFormat2aSubtree',
3380
3379
    )
3381
3380
 
3382
3381
 
3560
3559
        return InterRepository._same_model(source, target)
3561
3560
 
3562
3561
 
3563
 
class InterWeaveRepo(InterSameDataRepository):
3564
 
    """Optimised code paths between Weave based repositories.
3565
 
 
3566
 
    This should be in bzrlib/repofmt/weaverepo.py but we have not yet
3567
 
    implemented lazy inter-object optimisation.
3568
 
    """
3569
 
 
3570
 
    @classmethod
3571
 
    def _get_repo_format_to_test(self):
3572
 
        from bzrlib.repofmt import weaverepo
3573
 
        return weaverepo.RepositoryFormat7()
3574
 
 
3575
 
    @staticmethod
3576
 
    def is_compatible(source, target):
3577
 
        """Be compatible with known Weave formats.
3578
 
 
3579
 
        We don't test for the stores being of specific types because that
3580
 
        could lead to confusing results, and there is no need to be
3581
 
        overly general.
3582
 
        """
3583
 
        from bzrlib.repofmt.weaverepo import (
3584
 
                RepositoryFormat5,
3585
 
                RepositoryFormat6,
3586
 
                RepositoryFormat7,
3587
 
                )
3588
 
        try:
3589
 
            return (isinstance(source._format, (RepositoryFormat5,
3590
 
                                                RepositoryFormat6,
3591
 
                                                RepositoryFormat7)) and
3592
 
                    isinstance(target._format, (RepositoryFormat5,
3593
 
                                                RepositoryFormat6,
3594
 
                                                RepositoryFormat7)))
3595
 
        except AttributeError:
3596
 
            return False
3597
 
 
3598
 
    @needs_write_lock
3599
 
    def copy_content(self, revision_id=None):
3600
 
        """See InterRepository.copy_content()."""
3601
 
        # weave specific optimised path:
3602
 
        try:
3603
 
            self.target.set_make_working_trees(self.source.make_working_trees())
3604
 
        except (errors.RepositoryUpgradeRequired, NotImplemented):
3605
 
            pass
3606
 
        # FIXME do not peek!
3607
 
        if self.source._transport.listable():
3608
 
            pb = ui.ui_factory.nested_progress_bar()
3609
 
            try:
3610
 
                self.target.texts.insert_record_stream(
3611
 
                    self.source.texts.get_record_stream(
3612
 
                        self.source.texts.keys(), 'topological', False))
3613
 
                pb.update('Copying inventory', 0, 1)
3614
 
                self.target.inventories.insert_record_stream(
3615
 
                    self.source.inventories.get_record_stream(
3616
 
                        self.source.inventories.keys(), 'topological', False))
3617
 
                self.target.signatures.insert_record_stream(
3618
 
                    self.source.signatures.get_record_stream(
3619
 
                        self.source.signatures.keys(),
3620
 
                        'unordered', True))
3621
 
                self.target.revisions.insert_record_stream(
3622
 
                    self.source.revisions.get_record_stream(
3623
 
                        self.source.revisions.keys(),
3624
 
                        'topological', True))
3625
 
            finally:
3626
 
                pb.finished()
3627
 
        else:
3628
 
            self.target.fetch(self.source, revision_id=revision_id)
3629
 
 
3630
 
    @needs_read_lock
3631
 
    def search_missing_revision_ids(self, revision_id=None, find_ghosts=True):
3632
 
        """See InterRepository.missing_revision_ids()."""
3633
 
        # we want all revisions to satisfy revision_id in source.
3634
 
        # but we don't want to stat every file here and there.
3635
 
        # we want then, all revisions other needs to satisfy revision_id
3636
 
        # checked, but not those that we have locally.
3637
 
        # so the first thing is to get a subset of the revisions to
3638
 
        # satisfy revision_id in source, and then eliminate those that
3639
 
        # we do already have.
3640
 
        # this is slow on high latency connection to self, but as this
3641
 
        # disk format scales terribly for push anyway due to rewriting
3642
 
        # inventory.weave, this is considered acceptable.
3643
 
        # - RBC 20060209
3644
 
        if revision_id is not None:
3645
 
            source_ids = self.source.get_ancestry(revision_id)
3646
 
            if source_ids[0] is not None:
3647
 
                raise AssertionError()
3648
 
            source_ids.pop(0)
3649
 
        else:
3650
 
            source_ids = self.source._all_possible_ids()
3651
 
        source_ids_set = set(source_ids)
3652
 
        # source_ids is the worst possible case we may need to pull.
3653
 
        # now we want to filter source_ids against what we actually
3654
 
        # have in target, but don't try to check for existence where we know
3655
 
        # we do not have a revision as that would be pointless.
3656
 
        target_ids = set(self.target._all_possible_ids())
3657
 
        possibly_present_revisions = target_ids.intersection(source_ids_set)
3658
 
        actually_present_revisions = set(
3659
 
            self.target._eliminate_revisions_not_present(possibly_present_revisions))
3660
 
        required_revisions = source_ids_set.difference(actually_present_revisions)
3661
 
        if revision_id is not None:
3662
 
            # we used get_ancestry to determine source_ids then we are assured all
3663
 
            # revisions referenced are present as they are installed in topological order.
3664
 
            # and the tip revision was validated by get_ancestry.
3665
 
            result_set = required_revisions
3666
 
        else:
3667
 
            # if we just grabbed the possibly available ids, then
3668
 
            # we only have an estimate of whats available and need to validate
3669
 
            # that against the revision records.
3670
 
            result_set = set(
3671
 
                self.source._eliminate_revisions_not_present(required_revisions))
3672
 
        return self.source.revision_ids_to_search_result(result_set)
3673
 
 
3674
 
 
3675
 
class InterKnitRepo(InterSameDataRepository):
3676
 
    """Optimised code paths between Knit based repositories."""
3677
 
 
3678
 
    @classmethod
3679
 
    def _get_repo_format_to_test(self):
3680
 
        from bzrlib.repofmt import knitrepo
3681
 
        return knitrepo.RepositoryFormatKnit1()
3682
 
 
3683
 
    @staticmethod
3684
 
    def is_compatible(source, target):
3685
 
        """Be compatible with known Knit formats.
3686
 
 
3687
 
        We don't test for the stores being of specific types because that
3688
 
        could lead to confusing results, and there is no need to be
3689
 
        overly general.
3690
 
        """
3691
 
        from bzrlib.repofmt.knitrepo import RepositoryFormatKnit
3692
 
        try:
3693
 
            are_knits = (isinstance(source._format, RepositoryFormatKnit) and
3694
 
                isinstance(target._format, RepositoryFormatKnit))
3695
 
        except AttributeError:
3696
 
            return False
3697
 
        return are_knits and InterRepository._same_model(source, target)
3698
 
 
3699
 
    @needs_read_lock
3700
 
    def search_missing_revision_ids(self, revision_id=None, find_ghosts=True):
3701
 
        """See InterRepository.missing_revision_ids()."""
3702
 
        if revision_id is not None:
3703
 
            source_ids = self.source.get_ancestry(revision_id)
3704
 
            if source_ids[0] is not None:
3705
 
                raise AssertionError()
3706
 
            source_ids.pop(0)
3707
 
        else:
3708
 
            source_ids = self.source.all_revision_ids()
3709
 
        source_ids_set = set(source_ids)
3710
 
        # source_ids is the worst possible case we may need to pull.
3711
 
        # now we want to filter source_ids against what we actually
3712
 
        # have in target, but don't try to check for existence where we know
3713
 
        # we do not have a revision as that would be pointless.
3714
 
        target_ids = set(self.target.all_revision_ids())
3715
 
        possibly_present_revisions = target_ids.intersection(source_ids_set)
3716
 
        actually_present_revisions = set(
3717
 
            self.target._eliminate_revisions_not_present(possibly_present_revisions))
3718
 
        required_revisions = source_ids_set.difference(actually_present_revisions)
3719
 
        if revision_id is not None:
3720
 
            # we used get_ancestry to determine source_ids then we are assured all
3721
 
            # revisions referenced are present as they are installed in topological order.
3722
 
            # and the tip revision was validated by get_ancestry.
3723
 
            result_set = required_revisions
3724
 
        else:
3725
 
            # if we just grabbed the possibly available ids, then
3726
 
            # we only have an estimate of whats available and need to validate
3727
 
            # that against the revision records.
3728
 
            result_set = set(
3729
 
                self.source._eliminate_revisions_not_present(required_revisions))
3730
 
        return self.source.revision_ids_to_search_result(result_set)
3731
 
 
3732
 
 
3733
3562
class InterDifferingSerializer(InterRepository):
3734
3563
 
3735
3564
    @classmethod
3837
3666
                basis_id, delta, current_revision_id, parents_parents)
3838
3667
            cache[current_revision_id] = parent_tree
3839
3668
 
3840
 
    def _fetch_batch(self, revision_ids, basis_id, cache, a_graph=None):
 
3669
    def _fetch_batch(self, revision_ids, basis_id, cache):
3841
3670
        """Fetch across a few revisions.
3842
3671
 
3843
3672
        :param revision_ids: The revisions to copy
3844
3673
        :param basis_id: The revision_id of a tree that must be in cache, used
3845
3674
            as a basis for delta when no other base is available
3846
3675
        :param cache: A cache of RevisionTrees that we can use.
3847
 
        :param a_graph: A Graph object to determine the heads() of the
3848
 
            rich-root data stream.
3849
3676
        :return: The revision_id of the last converted tree. The RevisionTree
3850
3677
            for it will be in cache
3851
3678
        """
3919
3746
        if root_keys_to_create:
3920
3747
            root_stream = _mod_fetch._new_root_data_stream(
3921
3748
                root_keys_to_create, self._revision_id_to_root_id, parent_map,
3922
 
                self.source, graph=a_graph)
 
3749
                self.source)
3923
3750
            to_texts.insert_record_stream(root_stream)
3924
3751
        to_texts.insert_record_stream(from_texts.get_record_stream(
3925
3752
            text_keys, self.target._format._fetch_order,
3982
3809
        cache[basis_id] = basis_tree
3983
3810
        del basis_tree # We don't want to hang on to it here
3984
3811
        hints = []
3985
 
        if self._converting_to_rich_root and len(revision_ids) > 100:
3986
 
            a_graph = _mod_fetch._get_rich_root_heads_graph(self.source,
3987
 
                                                            revision_ids)
3988
 
        else:
3989
 
            a_graph = None
 
3812
        a_graph = None
3990
3813
 
3991
3814
        for offset in range(0, len(revision_ids), batch_size):
3992
3815
            self.target.start_write_group()
3994
3817
                pb.update('Transferring revisions', offset,
3995
3818
                          len(revision_ids))
3996
3819
                batch = revision_ids[offset:offset+batch_size]
3997
 
                basis_id = self._fetch_batch(batch, basis_id, cache,
3998
 
                                             a_graph=a_graph)
 
3820
                basis_id = self._fetch_batch(batch, basis_id, cache)
3999
3821
            except:
4000
3822
                self.source._safe_to_return_from_cache = False
4001
3823
                self.target.abort_write_group()
4067
3889
            basis_id = first_rev.parent_ids[0]
4068
3890
            # only valid as a basis if the target has it
4069
3891
            self.target.get_revision(basis_id)
4070
 
            # Try to get a basis tree - if its a ghost it will hit the
 
3892
            # Try to get a basis tree - if it's a ghost it will hit the
4071
3893
            # NoSuchRevision case.
4072
3894
            basis_tree = self.source.revision_tree(basis_id)
4073
3895
        except (IndexError, errors.NoSuchRevision):
4078
3900
 
4079
3901
InterRepository.register_optimiser(InterDifferingSerializer)
4080
3902
InterRepository.register_optimiser(InterSameDataRepository)
4081
 
InterRepository.register_optimiser(InterWeaveRepo)
4082
 
InterRepository.register_optimiser(InterKnitRepo)
4083
3903
 
4084
3904
 
4085
3905
class CopyConverter(object):
4273
4093
                is_resume = False
4274
4094
            try:
4275
4095
                # locked_insert_stream performs a commit|suspend.
4276
 
                return self._locked_insert_stream(stream, src_format, is_resume)
 
4096
                missing_keys = self.insert_stream_without_locking(stream,
 
4097
                                    src_format, is_resume)
 
4098
                if missing_keys:
 
4099
                    # suspend the write group and tell the caller what we is
 
4100
                    # missing. We know we can suspend or else we would not have
 
4101
                    # entered this code path. (All repositories that can handle
 
4102
                    # missing keys can handle suspending a write group).
 
4103
                    write_group_tokens = self.target_repo.suspend_write_group()
 
4104
                    return write_group_tokens, missing_keys
 
4105
                hint = self.target_repo.commit_write_group()
 
4106
                to_serializer = self.target_repo._format._serializer
 
4107
                src_serializer = src_format._serializer
 
4108
                if (to_serializer != src_serializer and
 
4109
                    self.target_repo._format.pack_compresses):
 
4110
                    self.target_repo.pack(hint=hint)
 
4111
                return [], set()
4277
4112
            except:
4278
4113
                self.target_repo.abort_write_group(suppress_errors=True)
4279
4114
                raise
4280
4115
        finally:
4281
4116
            self.target_repo.unlock()
4282
4117
 
4283
 
    def _locked_insert_stream(self, stream, src_format, is_resume):
 
4118
    def insert_stream_without_locking(self, stream, src_format,
 
4119
                                      is_resume=False):
 
4120
        """Insert a stream's content into the target repository.
 
4121
 
 
4122
        This assumes that you already have a locked repository and an active
 
4123
        write group.
 
4124
 
 
4125
        :param src_format: a bzr repository format.
 
4126
        :param is_resume: Passed down to get_missing_parent_inventories to
 
4127
            indicate if we should be checking for missing texts at the same
 
4128
            time.
 
4129
 
 
4130
        :return: A set of keys that are missing.
 
4131
        """
 
4132
        if not self.target_repo.is_write_locked():
 
4133
            raise errors.ObjectNotLocked(self)
 
4134
        if not self.target_repo.is_in_write_group():
 
4135
            raise errors.BzrError('you must already be in a write group')
4284
4136
        to_serializer = self.target_repo._format._serializer
4285
4137
        src_serializer = src_format._serializer
4286
4138
        new_pack = None
4326
4178
                # required if the serializers are different only in terms of
4327
4179
                # the inventory.
4328
4180
                if src_serializer == to_serializer:
4329
 
                    self.target_repo.revisions.insert_record_stream(
4330
 
                        substream)
 
4181
                    self.target_repo.revisions.insert_record_stream(substream)
4331
4182
                else:
4332
4183
                    self._extract_and_insert_revisions(substream,
4333
4184
                        src_serializer)
4366
4217
            # cannot even attempt suspending, and missing would have failed
4367
4218
            # during stream insertion.
4368
4219
            missing_keys = set()
4369
 
        else:
4370
 
            if missing_keys:
4371
 
                # suspend the write group and tell the caller what we is
4372
 
                # missing. We know we can suspend or else we would not have
4373
 
                # entered this code path. (All repositories that can handle
4374
 
                # missing keys can handle suspending a write group).
4375
 
                write_group_tokens = self.target_repo.suspend_write_group()
4376
 
                return write_group_tokens, missing_keys
4377
 
        hint = self.target_repo.commit_write_group()
4378
 
        if (to_serializer != src_serializer and
4379
 
            self.target_repo._format.pack_compresses):
4380
 
            self.target_repo.pack(hint=hint)
4381
 
        return [], set()
 
4220
        return missing_keys
4382
4221
 
4383
4222
    def _extract_and_insert_inventory_deltas(self, substream, serializer):
4384
4223
        target_rich_root = self.target_repo._format.rich_root_data
4441
4280
        """Create a StreamSource streaming from from_repository."""
4442
4281
        self.from_repository = from_repository
4443
4282
        self.to_format = to_format
 
4283
        self._record_counter = RecordCounter()
4444
4284
 
4445
4285
    def delta_on_metadata(self):
4446
4286
        """Return True if delta's are permitted on metadata streams.