Closed Bug 1170625 Opened 9 years ago Closed 9 years ago

Fetch popularity/trending in a single query when indexing

Categories

(Marketplace Graveyard :: Code Quality, defect, P2)

Product:
Marketplace Graveyard
Component:
Code Quality
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mat, Assigned: mat)

Details

When indexing apps/websites, we need to add the popularity/trending data for each region for this object to ES. We do that by making one query per region however, so because we now have all regions thanks to bug 1058257 that's ~ 250 x 2 queries per object we are indexing, which is pretty bad. It should happen in a single query for trending, and another for popularity.
Fixed in https://github.com/mozilla/zamboni/commit/9938240a2f624341ffd20a22753c3bd199bc2e5d

QA:
- Look at the popular page, note the apps order
- Make a change in devhub to some of these apps
- Make sure the changes are shown when you search for those apps a couple minutes later
- Make sure the popular page order has not changed
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.