Page MenuHomePhabricator

clindex: remove broken caches if things go wrong
ClosedPublic

Authored by quark on Dec 15 2017, 7:47 PM.

Details

Reviewers
phillco
Group Reviewers
Restricted Project
Commits
rFBHGX519dec04ea26: clindex: remove broken caches if things go wrong
Summary

Previously, nodemap or childmap will be removed if they are broken at load
time, but not runtime. That means with verify=1, the repo will stay in a
broken state forever if either cache is broken. This patch removes the cache
files if things go wrong at runtime.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

quark created this revision.Dec 15 2017, 7:47 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 15 2017, 7:47 PM
phillco accepted this revision.Dec 20 2017, 5:37 PM
phillco added a subscriber: phillco.

lgtm

This revision is now accepted and ready to land.Dec 20 2017, 5:37 PM
This revision was automatically updated to reflect the committed changes.