[MERGE] Remove knit caches

Aaron Bentley aaron.bentley at utoronto.ca
Mon Jun 26 00:21:05 BST 2006


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

Hi all,

Here's a patch that eliminates knit caches.  Theoretically, they could
speed up repeated reads or requests for multiple versions of a file, but
those operations are already fast, now that we have operations like
get_texts.

Knit caches don't provide a significant performance benefit for commit
operations-- the commit benchmark shows a 1% performance degradation
that may just be noise.

The log_verbose benchmark shows a 1% performance degradation, which
again is probably noise.

There aren't many operations that request multiple versions of a file.
Annotate gets multiple revisions.  It shows a 12% slowdown, but it
hasn't been optimized by using Repository.get_revisions

Even if the performance gains were more compelling, this would still be
a choice between performance and correctness, because the caches consume
an unlimited amount of memory.

Therefore, I propose that we get rid of the knit caches.  It seems to be
best to do them at a higher level, instead.

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

iD8DBQFEnxph0F+nu1YWqI0RAgk+AJ9S2QaW/AxBbh3GUxuaKm7gQCevzwCfVQD6
cazQ9DGYGKoOnqS9ShBUfKE=
=28J1
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: no-knit-caches.patch
Type: text/x-patch
Size: 8305 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060625/bb20cfaf/attachment.bin 


More information about the bazaar mailing list