[PATCH] osutil: introduce a posixfile type for Unix

Bryan O'Sullivan bos at serpentine.com
Thu Apr 12 18:10:54 CDT 2012


On Thu, Apr 12, 2012 at 2:36 PM, Idan Kamara <idankk86 at gmail.com> wrote:

>
> All these performance improvements are pretty neat but you have to
> wonder whether the shift to C and the inevitable code complexity they
> introduce are worth it in the long run.
>

That's certainly a concern, but these are all very modular changes. You
don't need to understand any of them in order to use them. Notice that the
posixfile change requires no change to any of the rest of the Mercurial
code base, except for the one spot that switches from file to posixfile if
available.

But in general, various parts of Mercurial are creaking under the load of
larger repositories, and the other patches I've been posting all help
performance quite a bit. The posixfile class is in a way the least
interesting of the lot, since it yields only a 10% improvement.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20120412/20f6d818/attachment.html>


More information about the Mercurial-devel mailing list