Ok, more detailed follow up:

Packagemanager.py:
Lines 1314-1320: don't these lines mean that if I have chosen a repo other then my default repo, select some packages from that repo which are also present in my default repo, then click install, I'll do the install from my default repo anyway? Since only the pkg stem is being passed in, I'm not sure how the authority info could be passed along. Does this also mean I can't select 3 pkgs from contrib and 2 from dev and have them all installed in one shot?

I don't totally grok all the other changes, but those were the things that jumped out at me.

Also, from bug 6365, it sounds like this is going to be a temporary fix to get find-as-you-type working till something else which scales is found? (If that's the case, I'm not sure that saying this fixes 6365 and closing it makes a lot of sense unless you're planning to open another to track the future scaling issues.) I also have some ideas on how we might pull a feature like this out of the search API, if you're interested let me know soon as that wad's starting to come together, and if that's going to be something we need, I want to make sure the design I have in mind at least allows for it going forward.

Other than 6365, I don't get why 6516 is needed, or even desired. Could you please fill out that bug with more explanation? Is it to fix 6227? If so, I don't think I understand how that helps.

Also, in the bug reports, I'm seeing a lot of reference to caching, but I don't really understand/know what that means in this context. I apologize if I've missed or forgotten the relevant discussions, but could you give an outline of what caching you have in mind? It might help me put these other changes in context.

Thanks,
Brock

Michal Pryc wrote:
Hello,
The webrev is:
http://cr.opensolaris.org/~migi/lists_split_10_02_2009_v1

This change makes the per repository underlying data model allowing selecting packages in many of them at the same time. Before the change, the main data model contaied all packages and it's selections. Current implementation splits that, so the selections are stored in dictionaries and data model is per repository only stored in the gtk.ListStore. This improves performance of many functions, as we are working on smaller data sets and we are not iterating through all packages to get selections.

Switching repositories may take now more time, but this will be improved a lot once caching will go in, but I didn't wanted to introduce
to many changes in one webrev.

This fixes bugs:

[Split the repository data models]:
http://defect.opensolaris.org/bz/show_bug.cgi?id=6516

[Package manager should elide categories which are empty]
http://defect.opensolaris.org/bz/show_bug.cgi?id=5662

[packagemanager chews CPU when changing repository]
http://defect.opensolaris.org/bz/show_bug.cgi?id=6227

[Unnecessary processing during PackageManager startup]
http://defect.opensolaris.org/bz/show_bug.cgi?id=5866

[Two entries appear in packagemanager for installed package]
From the bug description I think I was not able to reproduce, but I didn't understand
this bug description in 100%.
http://defect.opensolaris.org/bz/show_bug.cgi?id=6226

["find as you type" performance doesn't scale]
This is just an improvement for this bug. Having per repository data model refilter function will reduce time to perform it, but only in some cases. Please read more in the bug description.
http://defect.opensolaris.org/bz/show_bug.cgi?id=6365


_______________________________________________
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to