Merge lp:~stub/launchpad/update-storm into lp:launchpad
Proposed by
Stuart Bishop
Status: | Merged | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Merge reported by: | Stuart Bishop | ||||||||||||||||||||
Merged at revision: | not available | ||||||||||||||||||||
Proposed branch: | lp:~stub/launchpad/update-storm | ||||||||||||||||||||
Merge into: | lp:launchpad | ||||||||||||||||||||
Diff against target: | None lines | ||||||||||||||||||||
To merge this branch: | bzr merge lp:~stub/launchpad/update-storm | ||||||||||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Lange (community) | Approve | ||
Christian Reis | release-critical | Pending | |
Review via email: mp+9130@code.launchpad.net |
To post a comment you must log in.
Bug #393625 was raised because one of our scripts was using an unacceptable amount of RAM. Testing showed reducing the Storm cache size fixed the problem. Most likely, this script is dealing with objects containing long text columns.
This branch changes the default Storm cache size to be much more conservative. It also adds a load of boilerplate so we can easily override the conservative default for individual components where we detect slowness due to cache thrashing and can afford to use more RAM.
The appserver cache size settings remain unchanged from the value we have been running on edge for the last few weeks.
I would like to get this branch landed release critical so we don't have to cherry pick or disable update-pkgcache.