[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