I had a few questions and I sat with Michal and got answers for them.

Q. What changes were made for 4341?
A. The use of state is get_image_from_directory ensure that GUI results 
are the same as CLI results

Q. How is the same package being in more than one authority handled?
A. The use of pkg_stem ensures that we have separate rows for the two 
packages.

One minor nit:
The assignment of default_auth in update_package_list is not necesary as 
it is unused.

Padraig

 
On 12/05/08 11:31, Michal Pryc wrote:
> Hello,
> The webrev is at:
> http://cr.opensolaris.org/~migi/05_12_2008_bug5645_5289_4341_5150/
>
> Fixes:
> 1.
>   http://defect.opensolaris.org/bz/show_bug.cgi?id=5645
>   Package Manager should have busy cursor for certain actions
>
> 2.
>   http://defect.opensolaris.org/bz/show_bug.cgi?id=5289
>   Package Manager do not respect the same packages from non default 
> authorities
>
> 3.
>   http://defect.opensolaris.org/bz/show_bug.cgi?id=4341
>   IPS GUI and CLI list different status for some packages
>
> 4.
>   http://defect.opensolaris.org/bz/show_bug.cgi?id=5150
>   PM loading of package entries takes too long
>
> The 4. is partialy fixed. The time is reduced on my machine from:
>
> After __init__ :       0.31704211235
> Call to load_catalogs: 10.3101949692
> Enter get_list func:   10.3106389046
> After call to sort:    17.8955469131
> Return get_list func:  39.3143429756
>
> As we can see the total time before was 39 seconds for returing from 
> the get_list func which is basically the last func for startup.
>
> The new times (proposed in this webrev):
>
> After __init__:        0.258629083633
> Call to load_catalogs: 10.1712219715
> Enter get_list func:   10.171653986
> After call to sort:    15.8203570843
> Return get_list func:  17.8212211132
>
> As we can see we are up and running in 17.8 sec which is ~45% of the 
> time spend before improvements. The bottle neck is the call to 
> img.load_catalogs() which is more then 60% of the time for the startup.
>
> best
> Michal
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to