Bug 451083
Summary: | pkgorder speed improvement | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | Dennis Gregorovic <dgregor> | ||||
Component: | anaconda | Assignee: | Martin Gracik <mgracik> | ||||
Status: | CLOSED ERRATA | QA Contact: | Alexander Todorov <atodorov> | ||||
Severity: | low | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 5.2 | CC: | atodorov, borgan, ddumas, dmach | ||||
Target Milestone: | rc | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2009-09-02 09:55:50 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Dennis Gregorovic
2008-06-12 18:07:31 UTC
Created attachment 309118 [details]
remove repeated glob.glob calls
This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. Removed the glob.glob calls from printMatchingPkgs() function. Now the script builds the package list just once outside of the function, and then uses fnmatch in the function to print the matching packages. Tested on a custom repo, and it gives the same results as the original script, in 1/2 the time. Will be available in version 11.1.2.169. with anaconda-runtime-11.1.2.172-1 real 5m30.054s user 5m22.683s sys 0m7.009s compared to real 6m5.938s user 5m38.206s sys 0m27.373s on the same package set using pkgorder from 5.3. can we move this one to verified ? Dennis, can you provide some data from pkgorder execution? My test in comment #6 doesn't show big difference so I'm not sure if we want to move this to VERIFIED. Thanks! For the 5.4 ppc Server tree with 5.4 anaconda, I get: real 6m39.149s user 5m7.330s sys 0m31.468s For the same tree with the 5.3 anaconda, I get: real 9m24.324s user 6m26.870s sys 1m22.537s So, I'm seeing about a 25% improvement. I think we can close this bz and open a new one if we want to make other performance improvements. Moving to VERIFIED as per comment #8 An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on therefore solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2009-1306.html |