Merge lp:~seif/zeitgeist/optimize-find-related-uris into lp:zeitgeist/0.1
Proposed by
Seif Lotfy
Status: | Merged |
---|---|
Merge reported by: | Seif Lotfy |
Merged at revision: | not available |
Proposed branch: | lp:~seif/zeitgeist/optimize-find-related-uris |
Merge into: | lp:zeitgeist/0.1 |
Diff against target: |
29 lines (+10/-7) 1 file modified
_zeitgeist/engine/main.py (+10/-7) |
To merge this branch: | bzr merge lp:~seif/zeitgeist/optimize-find-related-uris |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Seif Lotfy | Approve | ||
Review via email: mp+38820@code.launchpad.net |
Description of the change
This is nothing else but a tiny optimization. Instead of calling get_events for the ids and thus resulting in having "Events" returned I did manual querying of the DB to get timestamp, id and uri. Thus saving us time and memory. The results are 2x faster than before
To post a comment you must log in.
RainCT gave me an ACK on IRC /code.edge. launchpad. net/~seif/ zeitgeist/ optimize- find-related- uris/+merge/ 38820
<seif_> thekorn, RainCT
<seif_> https:/
<RainCT> seif_: cool
<RainCT> seif_: go for it
<RainCT> (if the tests work)