[Yum-devel] [REGRESSION] Performance problem with SqliteSack and YumBase._getSacks changes

[Florian Festi]

Hi!

Current HEAD has significant performance problems. It turns out that the 
patch c65bdd8ffd3bfa1e6d29e904dda7dcd87538df6b "If the sack has no packages, 
we want to go back through and try to set it up" from Jeremy calls __len__ 
of MetaSack quite often. This results in calls to simplePkgList() which is 
no longer cached, due to the last few changes to the SqliteSack by James 
Antill. As a result my 1000 pkg F8 install drops from 25 to 270 seconds - 
just for depsolving.

I am a bit unsure which way to fix this disaster. I especially dislike that 
bool query against the pkgsack in YumBase._getSacks. But as the previous 
comparison with None doesn't cut it we probably need something completely 
different.

Comments?

Florian