Brock Pytlik wrote: > Webrev: > http://cr.opensolaris.org/~bpytlik/ips-4886-v1/ > > Bug: > http://defect.opensolaris.org/bz/show_bug.cgi?id=4886 > api.info shouldn't be reloading the catalog from disk needlessly > > For 10 repeated calls to api.info, this change reduces the time needed > from about 10 seconds to about 1.3 seconds. > > John, I think I'm seeing some speed up in the PackageManager but not on > the order of magnitude that the CLI shows. Can you give me more > information about how you performance tested the PackageManager to > determine that reading the catalog file was the main source of slow > down? It's also I was looking at the wrong use case, I thought this was > supposed to improve the time it took the details panel to be filled in. > Any feedback you could give from trying this patch would be appreciated. > > Thanks, > Brock > > _______________________________________________ > pkg-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >
Just a follow up to the performance notes for PackageManager, while I'm not sure there's a substantial difference from the users perspective, DTrace shows a massive improvement in speed. Using the gate to look at the first 7 packages in PackageManager meant we spent about 62 s in the api.info call, while with this fix, we spent 3.3 s in the api.info call. Overall, the time to look at the info for the 7 packages (after the program had finished it's initial loading) fell from 69s to 13s. Anecdotally, running without dtrace, looking at the info for the same 7 packages went from 30s to 12s. Brock _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
