Merge lp:~seif/zeitgeist/memory into lp:zeitgeist/0.1
Proposed by
Seif Lotfy
Status: | Merged |
---|---|
Merged at revision: | 1767 |
Proposed branch: | lp:~seif/zeitgeist/memory |
Merge into: | lp:zeitgeist/0.1 |
Diff against target: |
195 lines (+31/-20) 5 files modified
_zeitgeist/engine/datamodel.py (+1/-1) _zeitgeist/engine/main.py (+12/-11) _zeitgeist/engine/remote.py (+1/-1) _zeitgeist/engine/sql.py (+10/-0) test/engine-test.py (+7/-7) |
To merge this branch: | bzr merge lp:~seif/zeitgeist/memory |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Siegfried Gevatter | Approve | ||
Review via email: mp+63848@code.launchpad.net |
Description of the change
Reduce memory consumption by:
1) Using generators
2) disable SQL Cache (no real performance decline)
3) Use arrays for storing ids instead of lists
( 4) use tuples instead of lists when possible)
Results for this is less memory consumption. I think more can be done if we start using slots... But this is a clean hack without messing up the API/ABI
To post a comment you must log in.
Merged, but without the PRAGMA.