Page MenuHomePhabricator

remotefilelog: add linknode fixup logging

Authored by stash on Sep 28 2017, 12:58 PM.


Group Reviewers
Restricted Project
rFBHGX3971b56b4d5b: remotefilelog: add linknode fixup logging

The goal of the whole series is to have logging around linknode fixup. It's a slow operation, and we have two heuristics to make it faster. Unfortunately we have no idea about how well these heuristics actually help. This series of diffs aims to fix this problem. After this series is landed and logging is enabled, I hope that we can find out if we really these heuristics at all.

Let's log more data to make it easier to analyze where and how often does
linknode adjusing happen.

Test Plan

Run test-remotefilelog-linknodes.t

Diff Detail

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

Event Timeline

stash created this revision.Sep 28 2017, 12:58 PM
stash edited the summary of this revision. (Show Details)Sep 28 2017, 1:10 PM
quark accepted this revision.Sep 29 2017, 8:57 PM
quark added a subscriber: quark.

Make sure the cl change is done before landing.


Maybe these belong to shallowutils and getreponame can replace some existing code paths.


This could be a critical path and repo[x] overhead may need to be considered.

It's better to use:


to avoid the context overhead. That also uses "unfiltered" which is more desirable.

This revision is now accepted and ready to land.Sep 29 2017, 8:57 PM
stash updated this revision to Diff 2204.Sep 30 2017, 2:45 PM

addressed comments

stash updated this revision to Diff 2205.Sep 30 2017, 3:14 PM

add tests

This revision was automatically updated to reflect the committed changes.