[MERGE/RFC] Shortcut 'iter_references' when we can't have references

Aaron Bentley aaron at aaronbentley.com
Wed Jun 18 21:25:10 BST 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Ian Clatworthy wrote:
> John Arbash Meinel wrote:
>> In playing around with merging, I found that the "WT.iter_references()"
>> call was showing up somewhat heavily in the time for doing a merge.
> 
> I found the same thing when profiling branch on large projects.
> In the case of branch though, IIRC the slow iter_references call was
> being done in bzrdir.sprout() and the method was on DirStateRevisionTree,
> not workingtree_4.

Strangely, I didn't have to include that change to get equivalent
performance to your patch.

>> Thoughts? It seems a shame to penalize (heavily) our normal code because
>> we might be supporting references in some formats (that aren't even
>> considered production worthy yet.)
> 
> That was my thinking too. I'm not close enough to the subtree-related
> code to know how safe making this change is

I think it's a good idea.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFIWW8m0F+nu1YWqI0RAvB5AJwNLqpjGL4KbLu+Ye+SuFiJUM5fpwCeIhjd
FsApAWLdXE01cqrW9aGxOMM=
=oaje
-----END PGP SIGNATURE-----



More information about the bazaar mailing list