Mercurial > hg
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()')) |