##// END OF EJS Templates
emitrevision: consider ancestors revision to emit as available base...
emitrevision: consider ancestors revision to emit as available base This should make more delta base valid. This notably affects: * case where we skipped some parent with empty delta to directly delta against an ancestors * case where an intermediate snapshots is stored. This change means we could sent largish intermediate snapshots over the wire. However this is actually a sub goal here. Sending snapshots over the wire means the client have a high odd of simply storing the pre-computed delta instead of doing a lengthy process that will… end up doing the same intermediate snapshot. In addition the overall size of snapshot (or any level) is "only" some or the overall delta size. (0.17% for my mercurial clone, 20% for my clone of Mozilla try). So Sending them other the wire is unlikely to change large impact on the bandwidth used. If we decide that minimising the bandwidth is an explicit goal, we should introduce new logic to filter-out snapshot as delta. The current code has no notion explicite of snapshot so far, they just tended to fall into the wobbly filtering options. In some cases, this patch can yield large improvement to the bundling time: ### data-env-vars.name = mozilla-try-2019-02-18-zstd-sparse-revlog # benchmark.name = perf-bundle # benchmark.variants.revs = last-100000 before: 68.787066 seconds after: 47.552677 seconds (-30.87%) That translate to large improvement to the pull time : ### data-env-vars.name = mozilla-try-2019-02-18-zstd-sparse-revlog # benchmark.name = pull # benchmark.variants.issue6528 = disabled # benchmark.variants.revs = last-100000 before: 142.186625 seconds after: 75.897745 seconds (-46.62%) No significant negative impact have been observed.

File last commit:

r49730:6000f5b2 default
r50685:f463eb67 stable
Show More
tracing.py
61 lines | 1.6 KiB | text/x-python | PythonLexer
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 # Support code for event tracing in Mercurial. Lives in demandimport
# so it can also be used in demandimport.
#
# Copyright 2018 Google LLC.
#
# This software may be used and distributed according to the terms of the
# GNU General Public License version 2 or any later version.
import contextlib
import os
_pipe = None
_checked = False
demandimport: explicitly declare `_session` at the module level...
r42939 _session = 'none'
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290
Augie Fackler
formatting: blacken the codebase...
r43346
Augie Fackler
tracing: extract tracing-active logic to separate function...
r42676 def _isactive():
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 global _pipe, _session, _checked
if _pipe is None:
if _checked:
Augie Fackler
tracing: extract tracing-active logic to separate function...
r42676 return False
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 _checked = True
if 'HGCATAPULTSERVERPIPE' not in os.environ:
Augie Fackler
tracing: extract tracing-active logic to separate function...
r42676 return False
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 _pipe = open(os.environ['HGCATAPULTSERVERPIPE'], 'w', 1)
_session = os.environ.get('HGCATAPULTSESSION', 'none')
Augie Fackler
tracing: extract tracing-active logic to separate function...
r42676 return True
Augie Fackler
formatting: blacken the codebase...
r43346
Augie Fackler
tracing: extract tracing-active logic to separate function...
r42676 @contextlib.contextmanager
def log(whencefmt, *whenceargs):
if not _isactive():
yield
return
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 whence = whencefmt % whenceargs
try:
Augie Fackler
tracing: ignore any IOErrors when writing to pipe...
r39434 # Both writes to the pipe are wrapped in try/except to ignore
# errors, as we can see mysterious errors in here if the pager
# is active. Presumably other conditions could trigger
# problems too.
try:
_pipe.write('START %s %s\n' % (_session, whence))
except IOError:
pass
Augie Fackler
tracing: new module to make tracing events in hg easier...
r39290 yield
finally:
Augie Fackler
tracing: ignore any IOErrors when writing to pipe...
r39434 try:
_pipe.write('END %s %s\n' % (_session, whence))
except IOError:
pass
Augie Fackler
tracing: add support for emitting counters...
r42677
Augie Fackler
formatting: blacken the codebase...
r43346
Augie Fackler
tracing: add support for emitting counters...
r42677 def counter(label, amount, *labelargs):
if not _isactive():
return
l = label % labelargs
# See above in log() for why this is in a try/except.
try:
_pipe.write('COUNTER %s %d %s\n' % (_session, amount, l))
except IOError:
pass