[RFC][PERFORMANCE][PYREX][TIRED OF THESE YET?] *fast* walkdirs.

[Robert Collins]

On Fri, 2006-06-09 at 20:15 +0300, Alexander Belchenko wrote:
> Robert Collins пишет:
> > The attached patch combines two things: 
> >  * Proof of concept work for using pyrex for selected modules, to
> > provide a C version and a python version. Its likely that the C version
> > would be platform specific in some cases, and not in other.
> >  * A demonstration with pyrex of binding to readdir to get directory
> > type information. When available this provides a 50% speedup to the time
> > to walk the directory tree - even when the whole tree is in hot cache -
> > for the 10K kernel-like-tree scenario.
> > 
> > Obviously having C modules should be done with care given their
> > downsides - loops in the C module will ignore Ctrl-C unless you
> > explicitly check for it, bugs are harder to analyse, etc etc.
> 
> May I suggest to keep this new C-modules in separate branch, not in 
> bzr.dev until it goes mature?

Of course.

> Because having C-modules require to compiling it and this is create some 
> difficulties on windows (where C-compilers is not default part of 
> installation).
> 
> As compromise I suggest to make new C-extensions optional and give a 
> chance to bzr to works even without it. I did not look at your patch so 
> probably you already provide this option.

Have a look ;).

> Or new phase of bzr development means that 0.8.2 is latest stable bzr 
> version for windows users (without having pain with compiling 
> C-extension) for some time (probably months)?

I think we will have a new version soon that is faster, and better for
windows users. It should not *require* a C compiler for anyone.

Rob
-- 
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060610/d8c08dad/attachment.pgp