Closed Bug 912097 Opened 11 years ago Closed 9 years ago

Switch Fireplace app resource to fetch from ES

Categories

(Marketplace Graveyard :: API, defect, P4)

Product:
Marketplace Graveyard
Component:
API
Platform:
x86
macOS
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: ashort, Unassigned)

References

Details

(Whiteboard: [repoman])

Instead of hitting the DB for fetching apps for consumer pages, let's use ES -- we already have an app_to_dict function for this.
Priority: -- → P2
Depends on: 945820
Blocks: 926640
Assignee: nobody → mpillard
This is trivial to do now that everything has been properly DRF-ified but I don't know if it's that useful. AppViewSet is already doing pretty well from a performance standpoint: http://dashboard.mktadm.ops.services.phx1.mozilla.com/graphite-api?site=marketplace&graph=fireplace.api.appviewset Decreasing priority and un-assigning myself for now.
Assignee: mpillard → nobody
Priority: P2 → P3
Whiteboard: [repoman]
Priority: P3 → P4
I think this is still valuable to keep everything consistent. Let's do it.
Unlikely to ever be implemented by us and not something that can easily be picked up by a contributor. Also not very useful per comment 1 (and we still need a SQL-based AppViewSet for the write operations).
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.