[Yum-devel] [PATCH] Big speed boost for depsolving "large" transactions.

Seth Vidal skvidal at fedoraproject.org
Mon Nov 2 18:29:20 UTC 2009



On Mon, 2 Nov 2009, James Antill wrote:

> This follows on from the email I sent last week, about perf. notes on
> loading prco data, specifically:
>
> """
> But I couldn't see anything that made a worthwhile difference. It's
> possible some kind of "tsInfo local" PackageSack() would help, but maybe
> something for another day...
> """
>
> ...this implements that. And it's mostly awesome, except for the fact
> that we don't have a pkgKey index on files atm. so pkg.simpleFiles()
> is _really_ slow.
> We have 3 options AFAICS (in my order of "good"):
>
> 1. Create the index in yum, if needed.
>
> 2. Don't bother with the patch, at least until a new y-m-p is out with the
>   new index.
>
> 3. Always load filelists, at least until a new y-m-p is out.
>
> ...this patch goes with #1.
>

After some discussion on irc I think the plan is:

1. deprecate the simpleFiles() method since what it DOES and what it was 
supposed to do is ambiguous. Additionally b/c it returns two different 
thing depending on the type of package object it is :(

2. add a new method to do the above and/or modify returnFileEntries()

3. ACK on the rest.


-sv



More information about the Yum-devel mailing list