D4274: manifest: remove addgroup() from manifestlog and imanifestlog
indygreg (Gregory Szorc)
phabricator at mercurial-scm.org
Wed Aug 22 12:37:58 EDT 2018
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGaad4b46e89bb: manifest: remove addgroup() from manifestlog and imanifestlog (authored by indygreg, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D4274?vs=10346&id=10522
REVISION DETAIL
https://phab.mercurial-scm.org/D4274
AFFECTED FILES
mercurial/changegroup.py
mercurial/manifest.py
mercurial/repository.py
CHANGE DETAILS
diff --git a/mercurial/repository.py b/mercurial/repository.py
--- a/mercurial/repository.py
+++ b/mercurial/repository.py
@@ -1032,22 +1032,6 @@
Raises ``error.LookupError`` if the node is not known.
"""
- def addgroup(deltas, linkmapper, transaction):
- """Process a series of deltas for storage.
-
- ``deltas`` is an iterable of 7-tuples of
- (node, p1, p2, linknode, deltabase, delta, flags) defining revisions
- to add.
-
- The ``delta`` field contains ``mpatch`` data to apply to a base
- revision, identified by ``deltabase``. The base node can be
- ``nullid``, in which case the header from the delta can be ignored
- and the delta used as the fulltext.
-
- Returns a list of nodes that were processed. A node will be in the list
- even if it existed in the store previously.
- """
-
class completelocalrepository(interfaceutil.Interface):
"""Monolithic interface for local repositories.
diff --git a/mercurial/manifest.py b/mercurial/manifest.py
--- a/mercurial/manifest.py
+++ b/mercurial/manifest.py
@@ -1485,9 +1485,6 @@
def rev(self, node):
return self._revlog.rev(node)
- def addgroup(self, deltas, linkmapper, transaction):
- return self._revlog.addgroup(deltas, linkmapper, transaction)
-
@interfaceutil.implementer(repository.imanifestrevisionwritable)
class memmanifestctx(object):
def __init__(self, manifestlog):
diff --git a/mercurial/changegroup.py b/mercurial/changegroup.py
--- a/mercurial/changegroup.py
+++ b/mercurial/changegroup.py
@@ -253,7 +253,8 @@
# be empty during the pull
self.manifestheader()
deltas = self.deltaiter()
- repo.manifestlog.addgroup(deltas, revmap, trp)
+ # TODO this violates storage abstraction.
+ repo.manifestlog._revlog.addgroup(deltas, revmap, trp)
prog.complete()
self.callback = None
To: indygreg, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list