Mercurial > hg
diff hgext/fsmonitor/__init__.py @ 34885:df2ff314e36f
fsmonitor: warn when fsmonitor could be used
fsmonitor can significantly speed up operations on large working
directories. But fsmonitor isn't enabled by default, so naive users
may not realize there is a potential to make Mercurial faster.
This commit introduces a warning to working directory updates when
fsmonitor could be used.
The following conditions must be met:
* Working directory is previously empty
* New working directory adds >= N files (currently 50,000)
* Running on Linux or MacOS
* fsmonitor not enabled
* Warning not disabled via config override
Because of the empty working directory restriction, most users will
only see this warning during `hg clone` (assuming very few users
actually do an `hg up null`).
The addition of a warning may be considered a BC change. However, clone
has printed warnings before. Until recently, Mercurial printed a warning
with the server's certificate fingerprint when it wasn't explicitly
trusted for example. The warning goes to stderr. So it shouldn't
interfere with scripts parsing meaningful output.
The OS restriction was on the advice of Facebook engineers, who only
feel confident with watchman's stability on the supported platforms.
.. feature::
Print warning when fsmonitor isn't being used on a large repository
Differential Revision: https://phab.mercurial-scm.org/D894
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Wed, 18 Oct 2017 22:57:15 +0200 |
parents | dacfcdd8b94e |
children | dbb542326582 |
line wrap: on
line diff
--- a/hgext/fsmonitor/__init__.py Fri Oct 06 06:48:43 2017 -0700 +++ b/hgext/fsmonitor/__init__.py Wed Oct 18 22:57:15 2017 +0200 @@ -18,6 +18,9 @@ repository as necessary. You'll need to install Watchman from https://facebook.github.io/watchman/ and make sure it is in your PATH. +fsmonitor is incompatible with the largefiles and eol extensions, and +will disable itself if any of those are active. + The following configuration options exist: :: @@ -58,9 +61,22 @@ that can outpace the IPC overhead of getting the result data for the full repo from Watchman. Defaults to false. -fsmonitor is incompatible with the largefiles and eol extensions, and -will disable itself if any of those are active. +:: + + [fsmonitor] + warn_when_unused = (boolean) + +Whether to print a warning during certain operations when fsmonitor would be +beneficial to performance but isn't enabled. +:: + + [fsmonitor] + warn_update_file_count = (integer) + +If ``warn_when_unused`` is set and fsmonitor isn't enabled, a warning will +be printed during working directory updates if this many files will be +created. ''' # Platforms Supported