diff mercurial/localrepo.py @ 32266:2b6692df1bdf

caches: stop warming the cache after 'localrepo.commitctx'

Now that we garantee that branchmap cache are updated at the end of the
transaction we can drop that one. This removes a problematic case with nested
transaction where the new cache could be written on disk before the transaction
is finished.

The test change is harmless, since we update the cache at a later point, the
dirstate have been updated in between.

--- a/mercurial/localrepo.py	Tue May 02 21:35:06 2017 +0200
+++ b/mercurial/localrepo.py	Tue May 02 18:56:07 2017 +0200
@@ -1794,7 +1794,6 @@
                 # if minimal phase was 0 we don't need to retract anything
                 phases.retractboundary(self, tr, targetphase, [n])
             tr.close()
-            branchmap.updatecache(self.filtered('served'))
             return n
         finally:
             if tr: