[Yum-devel] [PATCH][RFC] Optimizing SqliteSack

[seth vidal]

On Tue, 2007-12-18 at 13:39 +0100, Florian Festi wrote:
> OK, after some more tests I found some more tweaks to reduce both runtime 
> and memory usage. I'd like to postpone the LRU cache stuff a bit but get the 
> sqlitesack changes and the additional tweaks in fast.
> 
> They include:
> 
>   * SqliteSack
>    * Use pkgKey instead of pkgId
>    * Generate only one package object per package (Now fixed)
>    * Load only minimal tags per default.
>    * Create YumAvailablePackageSqlite._files and ._checksums on demand only

All of the above looks reasonable. I'm a little worried about api
artifacts and responsiveness in file-resolutions from the change to
_files but this all seems reasonable.



>   * Package
>    * Make Package.pkgtup a real tuple

How much testing has this portion of the patch seen? I ask b/c I
remembered trying to switch away from the pkgtup property before. IIRC,
the wheels fall off when you use the objects in interesting ways.

-sv