D7128: rust-dirstate-status: use fast-path even with fsmonitor and sparse extensions

Alphare (Raphaël Gomès) phabricator at mercurial-scm.org
Thu Oct 17 23:31:01 UTC 2019


Alphare created this revision.
Herald added subscribers: mercurial-devel, mjpieters.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  When I initially ran the tests on my series, there were test failures related
  to those extensions. Now that the initial series has landed, I felt like going
  back to those issues because people with performance issues will often want to
  use fsmonitor and sparse.
  
  Either because of flaky tests or because the series has changed so much, I
  can't seem to reproduce these issues... let's widen the scope of the fast-path.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D7128

AFFECTED FILES
  mercurial/dirstate.py

CHANGE DETAILS

diff --git a/mercurial/dirstate.py b/mercurial/dirstate.py
--- a/mercurial/dirstate.py
+++ b/mercurial/dirstate.py
@@ -1115,11 +1115,6 @@
         elif not match.always():
             # Matchers have yet to be implemented
             use_rust = False
-        # We don't yet have a mechanism for extensions
-        elif sparse.enabled:
-            use_rust = False
-        elif not getattr(self, "_fsmonitordisable", True):
-            use_rust = False
 
         if use_rust:
             (



To: Alphare, #hg-reviewers
Cc: mjpieters, mercurial-devel


More information about the Mercurial-devel mailing list