On Tue, Oct 2, 2012 at 2:07 PM, Matt Mackall <span dir="ltr"><<a href="mailto:mpm@selenic.com" target="_blank">mpm@selenic.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":1j8">On the other hand, the only thing pushing this particular change is<br>
performance, so I'm in no great hurry to make a switch.</div></blockquote><div><br></div><div>I wrote a local patch that enables fncache2, and so far it's behaving itself.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":1j8"> I'd rather<br>
gather up a few more format improvements to go with it (for instance,<br>
dropping all the redundant data/ and .i in .hg/store/fncache).</div></blockquote></div><br><div>That doesn't sound like an improvement to me.</div><div><br></div><div>It would save a few bytes per entry, for a total space savings of 14% in the repos I looked at.</div>
<div><br></div><div>On the other hand, as far as I can tell, it would increase the amount of work we have to do per entry forever after: we'd have to tack "data/" and ".i" back on, either when writing or reading the fncache file, or when looking entries up.</div>
<div><br></div><div>We already know that fncache performance is hugely important, so the bar for doing extra computation should be very high.</div>