If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Fetch popularity/trending in a single query when indexing

RESOLVED FIXED

Status

Marketplace
Code Quality
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mat, Assigned: mat)

Tracking

Points:
---

Details

(Assignee)

Description

2 years ago
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.
(Assignee)

Comment 1

2 years ago
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
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.