[PATCH STABLE] perf: add a way to benchmark `dirstate.status`
Pierre-Yves David
pierre-yves.david at ens-lyon.org
Mon Nov 4 10:15:21 UTC 2019
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david at octobus.net>
# Date 1572823004 -3600
# Mon Nov 04 00:16:44 2019 +0100
# Branch stable
# Node ID 618ae1b4dcbacbc379a4ec96b7391876b6503e86
# Parent c2c3ee8794dd3749f2053555250576039cb2f1f4
# EXP-Topic bench-status
# Available At https://dev.heptapod.net/octobus/mercurial-devel/
# hg pull https://dev.heptapod.net/octobus/mercurial-devel/ -r 618ae1b4dcba
perf: add a way to benchmark `dirstate.status`
Getting more details about time spend in this specific internal bit is
meaningful.
diff --git a/contrib/perf.py b/contrib/perf.py
--- a/contrib/perf.py
+++ b/contrib/perf.py
@@ -760,7 +760,10 @@ def perfannotate(ui, repo, f, **opts):
@command(
b'perfstatus',
- [(b'u', b'unknown', False, b'ask status to look for unknown files')]
+ [
+ (b'u', b'unknown', False, b'ask status to look for unknown files'),
+ (b'', b'dirstate', False, b'benchmark the internal dirstate call'),
+ ]
+ formatteropts,
)
def perfstatus(ui, repo, **opts):
@@ -776,7 +779,20 @@ def perfstatus(ui, repo, **opts):
# timer(lambda: sum(map(len, repo.dirstate.status(m, [], False, False,
# False))))
timer, fm = gettimer(ui, opts)
- timer(lambda: sum(map(len, repo.status(unknown=opts[b'unknown']))))
+ if opts[b'dirstate']:
+ dirstate = repo.dirstate
+ m = scmutil.matchall(repo)
+ unknown = opts[b'unknown']
+
+ def status_dirstate():
+ s = dirstate.status(
+ m, subrepos=[], ignored=False, clean=False, unknown=unknown
+ )
+ sum(map(len, s))
+
+ timer(status_dirstate)
+ else:
+ timer(lambda: sum(map(len, repo.status(unknown=opts[b'unknown']))))
fm.end()
diff --git a/tests/test-contrib-perf.t b/tests/test-contrib-perf.t
--- a/tests/test-contrib-perf.t
+++ b/tests/test-contrib-perf.t
@@ -248,6 +248,7 @@ perfstatus
$ hg perfrevset 'all()'
$ hg perfstartup
$ hg perfstatus
+ $ hg perfstatus --dirstate
$ hg perftags
$ hg perftemplating
$ hg perfvolatilesets
More information about the Mercurial-devel
mailing list