Patchwork [5,of,5] perf: disable revlogs clearing in `perftags` by default

login
register
mail settings
Submitter Boris Feld
Date Nov. 20, 2018, 7:18 p.m.
Message ID <32bd83a430145e807bb4.1542741489@localhost.localdomain>
Download mbox | patch
Permalink /patch/36676/
State Accepted
Headers show

Comments

Boris Feld - Nov. 20, 2018, 7:18 p.m.
# HG changeset patch
# User Boris Feld <boris.feld@octobus.net>
# Date 1542735864 0
#      Tue Nov 20 17:44:24 2018 +0000
# Node ID 32bd83a430145e807bb4713d1938715a3963c68b
# Parent  1799103c7a6fb140cdb7317c3e7b7cf93295edd0
# EXP-Topic perf-tags
# Available At https://bitbucket.org/octobus/mercurial-devel/
#              hg pull https://bitbucket.org/octobus/mercurial-devel/ -r 32bd83a43014
perf: disable revlogs clearing in `perftags` by default

This aligns things with what `perfbookmarks` does. I decided to disable the
revlogs clearing by default to focus on the core logic by default, ignoring
side effects.

If we prefer to emphasize the side effect, we can instead keep this on in
`perftags` and enable it by default in `perfbookmarks`.

Patch

diff --git a/contrib/perf.py b/contrib/perf.py
--- a/contrib/perf.py
+++ b/contrib/perf.py
@@ -532,7 +532,7 @@  def perfheads(ui, repo, **opts):
 
 @command(b'perftags', formatteropts+
         [
-            (b'', b'clear-revlogs', True, 'refresh changelog and manifest'),
+            (b'', b'clear-revlogs', False, 'refresh changelog and manifest'),
         ])
 def perftags(ui, repo, **opts):
     import mercurial.changelog