Merge lp:~sil/desktopcouch/dont-parse-all-the-changes into lp:desktopcouch
Proposed by
Stuart Langridge
Status: | Merged |
---|---|
Approved by: | Chad Miller |
Approved revision: | 143 |
Merge reported by: | Chad Miller |
Merged at revision: | not available |
Proposed branch: | lp:~sil/desktopcouch/dont-parse-all-the-changes |
Merge into: | lp:desktopcouch |
Diff against target: |
13 lines (+1/-4) 1 file modified
desktopcouch/records/server_base.py (+1/-4) |
To merge this branch: | bzr merge lp:~sil/desktopcouch/dont-parse-all-the-changes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chad Miller (community) | Approve | ||
Eric Casteleijn (community) | Approve | ||
Review via email: mp+22414@code.launchpad.net |
Description of the change
Don't parse all the _changes feed; work out update_seq from the DB info, so we never request the whole feed, which is doom when the whole feed is large.
To post a comment you must log in.
Makes sense, looks good, tests pass.