D2106: infinitepush: drop logic related to treemanifest extension

2018-03-30 Thread pulkit (Pulkit Goyal)
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG7fa00a6f8180: infinitepush: drop logic related to 
treemanifest extension (authored by pulkit, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2106?vs=5367=7421

REVISION DETAIL
  https://phab.mercurial-scm.org/D2106

AFFECTED FILES
  hgext/infinitepush/__init__.py
  hgext/infinitepush/bundleparts.py

CHANGE DETAILS

diff --git a/hgext/infinitepush/bundleparts.py 
b/hgext/infinitepush/bundleparts.py
--- a/hgext/infinitepush/bundleparts.py
+++ b/hgext/infinitepush/bundleparts.py
@@ -68,19 +68,6 @@
 advisoryparams=params.iteritems(),
 data=cg))
 
-try:
-treemod = extensions.find('treemanifest')
-mfnodes = []
-for node in outgoing.missing:
-mfnodes.append(('', repo[node].manifestnode()))
-
-# Only include the tree parts if they all exist
-if not repo.manifestlog.datastore.getmissing(mfnodes):
-parts.append(treemod.createtreepackpart(
-repo, outgoing, treemod.TREEGROUP_PARTTYPE2))
-except KeyError:
-pass
-
 return parts
 
 def _validaterevset(repo, revset, bookmark):
diff --git a/hgext/infinitepush/__init__.py b/hgext/infinitepush/__init__.py
--- a/hgext/infinitepush/__init__.py
+++ b/hgext/infinitepush/__init__.py
@@ -440,16 +440,6 @@
 cgpart.addparam('version', version)
 parts.append(cgpart)
 
-try:
-treemod = extensions.find('treemanifest')
-except KeyError:
-pass
-else:
-if treemod._cansendtrees(bundlerepo, outgoing.missing):
-treepart = treemod.createtreepackpart(bundlerepo, outgoing,
-  treemod.TREEGROUP_PARTTYPE2)
-parts.append(treepart)
-
 return parts
 
 def _getbundleroots(oldrepo, bundlerepo, bundlerevs):
@@ -858,7 +848,6 @@
 return
 
 pushop.stepsdone.add('changesets')
-pushop.stepsdone.add('treepack')
 if not pushop.outgoing.missing:
 pushop.ui.status(_('no changes found\n'))
 pushop.cgresult = 0
@@ -953,13 +942,6 @@
 
 handleallparts = repo.ui.configbool('infinitepush', 'storeallparts')
 
-partforwardingwhitelist = []
-try:
-treemfmod = extensions.find('treemanifest')
-partforwardingwhitelist.append(treemfmod.TREEGROUP_PARTTYPE2)
-except KeyError:
-pass
-
 bundler = bundle2.bundle20(repo.ui)
 cgparams = None
 with bundle2.partiterator(repo, op, unbundler) as parts:
@@ -987,7 +969,7 @@
 op.records.add(scratchbranchparttype + '_skipphaseheads',
True)
 else:
-if handleallparts or part.type in partforwardingwhitelist:
+if handleallparts:
 # Ideally we would not process any parts, and instead just
 # forward them to the bundle for storage, but since this
 # differs from previous behavior, we need to put it behind 
a



To: pulkit, #hg-reviewers, indygreg
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2106: infinitepush: drop logic related to treemanifest extension

2018-02-09 Thread pulkit (Pulkit Goyal)
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  treemanifest is another facebook's extension which is not in core. Let's drop
  the logic related to infinitepush extension which is now ported to core.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2106

AFFECTED FILES
  hgext/infinitepush/__init__.py
  hgext/infinitepush/bundleparts.py

CHANGE DETAILS

diff --git a/hgext/infinitepush/bundleparts.py 
b/hgext/infinitepush/bundleparts.py
--- a/hgext/infinitepush/bundleparts.py
+++ b/hgext/infinitepush/bundleparts.py
@@ -68,19 +68,6 @@
 advisoryparams=params.iteritems(),
 data=cg))
 
-try:
-treemod = extensions.find('treemanifest')
-mfnodes = []
-for node in outgoing.missing:
-mfnodes.append(('', repo[node].manifestnode()))
-
-# Only include the tree parts if they all exist
-if not repo.manifestlog.datastore.getmissing(mfnodes):
-parts.append(treemod.createtreepackpart(
-repo, outgoing, treemod.TREEGROUP_PARTTYPE2))
-except KeyError:
-pass
-
 return parts
 
 def _validaterevset(repo, revset, bookmark):
diff --git a/hgext/infinitepush/__init__.py b/hgext/infinitepush/__init__.py
--- a/hgext/infinitepush/__init__.py
+++ b/hgext/infinitepush/__init__.py
@@ -440,16 +440,6 @@
 cgpart.addparam('version', version)
 parts.append(cgpart)
 
-try:
-treemod = extensions.find('treemanifest')
-except KeyError:
-pass
-else:
-if treemod._cansendtrees(bundlerepo, outgoing.missing):
-treepart = treemod.createtreepackpart(bundlerepo, outgoing,
-  treemod.TREEGROUP_PARTTYPE2)
-parts.append(treepart)
-
 return parts
 
 def _getbundleroots(oldrepo, bundlerepo, bundlerevs):
@@ -858,7 +848,6 @@
 return
 
 pushop.stepsdone.add('changesets')
-pushop.stepsdone.add('treepack')
 if not pushop.outgoing.missing:
 pushop.ui.status(_('no changes found\n'))
 pushop.cgresult = 0
@@ -953,13 +942,6 @@
 
 handleallparts = repo.ui.configbool('infinitepush', 'storeallparts')
 
-partforwardingwhitelist = []
-try:
-treemfmod = extensions.find('treemanifest')
-partforwardingwhitelist.append(treemfmod.TREEGROUP_PARTTYPE2)
-except KeyError:
-pass
-
 bundler = bundle2.bundle20(repo.ui)
 cgparams = None
 with bundle2.partiterator(repo, op, unbundler) as parts:
@@ -987,7 +969,7 @@
 op.records.add(scratchbranchparttype + '_skipphaseheads',
True)
 else:
-if handleallparts or part.type in partforwardingwhitelist:
+if handleallparts:
 # Ideally we would not process any parts, and instead just
 # forward them to the bundle for storage, but since this
 # differs from previous behavior, we need to put it behind 
a



To: pulkit, #hg-reviewers
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel