Closed
Bug 745062
Opened 12 years ago
Closed 12 years ago
memcached backend doesn't filter tabs correctly
Categories
(Cloud Services Graveyard :: Server: Sync, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: rfkelly, Unassigned)
Details
(Whiteboard: [qa-])
Attachments
(1 file)
|
767 bytes,
patch
|
telliott
:
review+
|
Details | Diff | Splinter Review |
A simple typo/oversight in the memcached backend causes it not to apply query parameter filters like "newer" to the tabs collection. Attached patch should be self-explanatory. There's actually already a test for this as part of the functional suite, but `make test` doesn't run it against the memcached backend by default. We should fix the test suite to enable this, but that's a separate bug.
Attachment #614660 -
Flags: review?(telliott)
|
||
Updated•12 years ago
|
Attachment #614660 -
Flags: review?(telliott) → review+
|
Reporter | |
Comment 1•12 years ago
|
||
https://github.com/mozilla-services/server-syncstorage/commit/36c2dc8d77699c514fe6a44384a0feb9424a9a5d
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
Whiteboard: [qa-]
|
|
||
Comment 2•12 years ago
|
||
Code verified and passed functional tests of Sync 2.0 and AITC 1.0 for a current deploy to qa1.
Status: RESOLVED → VERIFIED
|
|
Reporter | |
Comment 3•12 years ago
|
||
Backported to sync1.1: http://hg.mozilla.org/services/server-storage/rev/ec7f28650fa8
|
|
||
Comment 4•12 years ago
|
||
OK. Will re-verify against sync 1.1.
|
||
Updated•1 year ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•