[Updated] D8408: nodemap: skip persistent nodemap warming for revlog not using it

[marmoute (Pierre-Yves David)]

Herald added a subscriber: mercurial-patches.
Closed by commit rHGb4537125eb3c: nodemap: skip persistent nodemap warming for revlog not using it (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/D8408?vs=21089&id=21203

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

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

AFFECTED FILES
  mercurial/revlogutils/nodemap.py

CHANGE DETAILS

diff --git a/mercurial/revlogutils/nodemap.py b/mercurial/revlogutils/nodemap.py
--- a/mercurial/revlogutils/nodemap.py
+++ b/mercurial/revlogutils/nodemap.py
@@ -112,6 +112,11 @@
     To be used for updating the nodemap on disk outside of a normal transaction
     setup (eg, `debugupdatecache`).
     """
+    if revlog._inline:
+        return  # inlined revlog are too small for this to be relevant
+    if revlog.nodemap_file is None:
+        return  # we do not use persistent_nodemap on this revlog
+
     notr = _NoTransaction()
     _persist_nodemap(notr, revlog)
     for k in sorted(notr._postclose):



To: marmoute, #hg-reviewers, pulkit, Alphare
Cc: mercurial-patches, mercurial-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mercurial-scm.org/pipermail/mercurial-patches/attachments/20200423/38204402/attachment.html>