Closed Bug 1127144 Opened 9 years ago Closed 9 years ago

Deal with regions in elastic search

Categories

(Marketplace Graveyard :: Search, defect, P3)

Avenir
x86
macOS
defect

Tracking

(Not tracked)

VERIFIED FIXED
2015-05-12

People

(Reporter: andy+bugzilla, Assigned: mat)

References

Details

Rob can add more details here, but I'm filing this so we don't forget. Basically the way we store data in ES is going to bloat if we add in every region. We need to be able to cope with that.
We store a region specific value for both popular and trending values. I don't think there's anything we can do about this exploding out to 250+ fields for each of popular/trending since we will need to sort on these.

But, we can reduce the "over the wire" size by using `.fields(...)` to choose the subset of fields we need ES to return to us when making queries. The popular and trending fields don't need to be in this list since we don't expose these in the API. I'm sure there are others.
While we are trying to reduce "over the wire" size, we could consider doing the same thing you propose for regions for translations. We could also have a general blacklist/whitelist, maybe stored in the serializers, to avoid requesting fields we know we are not going to need/expose.
Priority: -- → P4
Priority: P4 → P3
Target Milestone: --- → 2015-05-12
Assignee: nobody → mpillard
Status: NEW → ASSIGNED
Fixed in https://github.com/mozilla/zamboni/commit/927bad0ebc7ed1cd693a730bfdfc77e73a45d0c1

QA:
- Please make sure search, new, popular pages still work.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified as fixed for all fields from above
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.