( )⚙ D1410 perftweaks: support treedirstate maps

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D1410

perftweaks: support treedirstate maps
ClosedPublic

Authored by mbthomas on Nov 14 2017, 12:39 PM.
Tags
None
Subscribers
None

Details

Reviewers
quark
Group Reviewers
Restricted Project
Commits
rFBHGXcfc2dd303b17: perftweaks: support treedirstate maps
Summary

Treedirstate maps won't have an inner _map, so will never be logged by
perftweaks. The maps have an efficient size lookup, so it's OK to always
obtain their size.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Revision Contents
Changeset List

PathPackages
Mhgext3rd/perftweaks.py (6 lines)

Diff 3939

hgext3rd/perftweaks.py

pass pass
def _trackdirstatesizes(runcommand, lui, repo, *args): def _trackdirstatesizes(runcommand, lui, repo, *args):
res = runcommand(lui, repo, *args) res = runcommand(lui, repo, *args)
if repo is not None and repo.local(): if repo is not None and repo.local():
dirstate = repo.dirstate dirstate = repo.dirstate
# if the _map attribute is missing on the map, the dirstate was not # if the _map attribute is missing on the map, the dirstate was not
# loaded. # loaded.
if '_map' in vars(dirstate) and '_map' in vars(dirstate._map): if (('_map' in vars(dirstate) and '_map' in vars(dirstate._map)) or
map_ = dirstate._map._map ('treedirstate' in getattr(repo, 'requirements', set()))):
lui.log('dirstate_size', '', lui.log('dirstate_size', '',
dirstate_size=len(map_)) dirstate_size=len(dirstate._map))
return res return res
def _tracksparseprofiles(runcommand, lui, repo, *args): def _tracksparseprofiles(runcommand, lui, repo, *args):
res = runcommand(lui, repo, *args) res = runcommand(lui, repo, *args)
if repo is not None and repo.local(): if repo is not None and repo.local():
if util.safehasattr(repo, 'getactiveprofiles'): if util.safehasattr(repo, 'getactiveprofiles'):
profiles = repo.getactiveprofiles() profiles = repo.getactiveprofiles()
lui.log('sparse_profiles', '', lui.log('sparse_profiles', '',