[PATCH] revset: don't create changectx when matching for files

Durham Goode durham at fb.com
Fri Mar 20 14:43:49 CDT 2015

On 3/20/15 11:41 AM, Matt Mackall wrote:
> On Tue, 2015-03-17 at 10:51 -0700, Durham Goode wrote:
>> A more detailed breakdown of the profiling is here:
>> https://urldefense.proofpoint.com/v1/url?u=https://bpaste.net/show/1affea0af5e4&k=ZVNjlDMF0FElm4dQtryO4A%3D%3D%0A&r=pHOG6Hz51SkYmYr%2FxoTFzw%3D%3D%0A&m=JBD9noTgZxxkE1x0gleLTZvXo7v30tZZB0IIgGLjenw%3D%0A&s=4d6f5d76347e2a86763013a5b788bf968294e44fe363188c56b3d767186231b9
>> It looks like repoview introduces some overhead (simply switching to an
>> unfiltered repo gets us from 12s to 9.9s, despite this repo not having
>> any obsolescence) and the remaining time is in the changectx constructor.
> Perhaps we should fast-path that instead.
I talked with Pierre-Yves and it's non-trivial to fast path 
repoview.changelog when working with repositories that do have filtered 
changes (which will soon be all of ours).  So if the current work around 
is too hacky, we'll have to come back and revisit this when we have more 

More information about the Mercurial-devel mailing list