Port enormous, horrible search view to oedipus

VERIFIED FIXED in 2011-11-08

Status

support.mozilla.org
Search
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: erik, Assigned: willkg)

Tracking

unspecified
2011-11-08
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
Port search.views.search from the search.clients API to oedipus. Lose a little piece of your soul as you futilely attempt to reverse-engineer dozens of pieces of state into some kind of comprehensible specification. Descend into irredeemable despair as your mind buckles under the cleaving weight of a thousand tangled couplings.

6 years experience with Windows Metro required. Sparkly crown with rubies in it a plus. Competitive pay.

(Actually, it probably won't be nearly that bad. 5th-circle-level at worst.)
(Reporter)

Updated

7 years ago
Blocks: 687912
Takificationalyzing this one.  Will don heat-proof underwear before embarking.
Assignee: nobody → willkg
I'm fairly confident I can't finish this in the next day, so pushing it to 11-01-2011.
Target Milestone: --- → 2011-11-01
Bumping this to 2011-11-08.
Target Milestone: 2011-11-01 → 2011-11-08
(Reporter)

Comment 4

7 years ago
Ran into a performance problem where we'd be doing a DB hit for every matching result, not just every result displayed. We tuned oedipus to fix this. Now it will actually be better-performing that the current hand-written code, in that it will do a single DB query per model (at worst) per page for the result list rather than doing a separate query for each result displayed.
(Reporter)

Comment 5

7 years ago
master: http://github.com/jsocol/kitsune/commit/26302fe

QA: Just spot-check to make sure search form (including Advanced Search) still works. Most of it is covered by automated tests.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Verified search returns results
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.