comparison mercurial/obsolete.py @ 18271:67872e939945

performance: speedup computation of obsolete revisions In their current state, revset calls can be very costly as we test predicates on the entire repository. As obsolete computation is used by the "hidden" filter, it needs to be very fast. This changet drops the revset call in favor of direct testing of the phase of a changeset. Performance test on my Mercurial checkout - 19857 total changesets, - 1584 obsolete changesets, - 13310 obsolescence markers. Before: ! obsolete ! wall 0.047041 After: ! obsolete ! wall 0.004590 Performance test on a Mozilla central checkout: - 117293 total changesets, - 1 obsolete changeset, - 1 obsolescence marker. Before: ! obsolete ! wall 0.001539 After: ! obsolete ! wall 0.000017
author Pierre-Yves David <pierre-yves.david@ens-lyon.org>
date Fri, 04 Jan 2013 03:14:54 +0100
parents a464deecc9dd
children 9818f22785b7
comparison
equal deleted inserted replaced
18270:48deb483a8f8 18271:67872e939945
647 647
648 @cachefor('obsolete') 648 @cachefor('obsolete')
649 def _computeobsoleteset(repo): 649 def _computeobsoleteset(repo):
650 """the set of obsolete revisions""" 650 """the set of obsolete revisions"""
651 obs = set() 651 obs = set()
652 nm = repo.changelog.nodemap 652 getrev = repo.changelog.nodemap.get
653 getphase = repo._phasecache.phase
653 for node in repo.obsstore.successors: 654 for node in repo.obsstore.successors:
654 rev = nm.get(node) 655 rev = getrev(node)
655 if rev is not None: 656 if rev is not None and getphase(repo, rev):
656 obs.add(rev) 657 obs.add(rev)
657 return set(repo.revs('%ld - public()', obs)) 658 return obs
658 659
659 @cachefor('unstable') 660 @cachefor('unstable')
660 def _computeunstableset(repo): 661 def _computeunstableset(repo):
661 """the set of non obsolete revisions with obsolete parents""" 662 """the set of non obsolete revisions with obsolete parents"""
662 return set(repo.revs('(obsolete()::) - obsolete()')) 663 return set(repo.revs('(obsolete()::) - obsolete()'))