[Updated] D9754: persistent-nodemap: write down the sprint conclusion

[marmoute (Pierre-Yves David)]

Closed by commit rHG34c043237d2d: persistent-nodemap: write down the sprint conclusion (authored by marmoute).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D9754?vs=24886&id=24966

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D9754/new/

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

AFFECTED FILES
  mercurial/configitems.py

CHANGE DETAILS

diff --git a/mercurial/configitems.py b/mercurial/configitems.py
--- a/mercurial/configitems.py
+++ b/mercurial/configitems.py
@@ -1266,9 +1266,21 @@
     b'usestore',
     default=True,
 )
-# Right now, the only efficient implement of the nodemap logic is in Rust, so
-# the persistent nodemap feature needs to stay experimental as long as the Rust
-# extensions are an experimental feature.
+# Right now, the only efficient implement of the nodemap logic is in Rust,
+#
+# The case was discussed that the 5.6 sprint and the following was decided for
+# feature that have an optional fast implementation (and are a performance
+# regression in the others)
+#
+#   * If the fast implementation is not available, Mercurial will refuse to
+#     access repository that requires it. Pointing to proper documentation
+#
+#   * An option exist to lift that limitation and allow repository access.
+#
+#     Such access will emit a warning unless configured not to.
+#
+#   * When sufficiently mature, the feature can be enabled by default only for
+#   installation that supports it.
 coreconfigitem(
     b'format', b'use-persistent-nodemap', default=False, experimental=True
 )



To: marmoute, #hg-reviewers, Alphare, pulkit
Cc: mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20210116/78b68ad0/attachment-0002.html>