Closed Bug 950839 Opened 11 years ago Closed 11 years ago

Collections /api/:version/rocketfuel/collections/:id/reorder/ is too slow

Categories

(Marketplace Graveyard :: API, defect, P2)

Product:
Marketplace Graveyard
Component:
API
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2013-12-17

People

(Reporter: cvan, Assigned: mat)

References

(
URL
)

Details

https://marketplace.firefox.com/api/v1/rocketfuel/collections/62/reorder/ http://f.cl.ly/items/040A0P3i0N0j3u3Z0b0p/Screen%20Shot%202013-12-16%20at%2011.55.27%20AM.png 9.25s, 8.75s. I'm not sure what's going on here. Is this endpoint synchronous and we're blocked on an actual DB save before we return the response? POST /api/v1/rocketfuel/collections/62/reorder/?_user=hearcomestreble%40gmail.com%<CENSORED> HTTP/1.1 Host: marketplace.firefox.com Connection: keep-alive Content-Length: 162 Cache-Control: no-cache Pragma: no-cache Origin: https://marketplace.firefox.com User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/31.0.1650.63 Safari/537.36 Content-Type: application/json Accept: */* Referer: https://marketplace.firefox.com/curation/collection/home-spain-featured Accept-Encoding: gzip,deflate,sdch Accept-Language: en-US,en;q=0.8,fr;q=0.6 Cookie: lang="en-US\054en-US"; <REDACTED>;|12=Reviewer=Reviewer=1
Yes, as noted in my comment chuck pointed out, there a synchronous indexing that should be asynchronous instead. I'll fix that.
Assignee: nobody → mpillard
Status: NEW → ASSIGNED
Priority: -- → P2
good catch. thanks for filing & fixing y'all
Fixed in https://github.com/mozilla/zamboni/commit/37a092c8f94f9ed1882b21ecbc531bd9b4b33ae3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-12-17
You need to log in before you can comment on or make changes to this bug.