Closed
Bug 874588
Opened 12 years ago
Closed 12 years ago
Add back statsd timings for search
Categories
(Marketplace Graveyard :: Search, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
2013-05-30
People
(Reporter: robhudson, Assigned: robhudson)
References
Details
(Whiteboard: p=1 [qa-])
Newer elasticutils removed this. We can add it back by subclassing the `S` class and adding it in ourselves in the `raw` call.
For reference, here's the commit that removed it:
https://github.com/mozilla/elasticutils/commit/6ccf9dde3e52751852c7bc132fbd585473ce704b#L2L200
Assignee | ||
Comment 1•12 years ago
|
||
It was discussed in the search meeting that having both the "took" ES timing and also a timing that covers request/response time would be useful.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → robhudson.mozbugs
Target Milestone: --- → 2013-05-30
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Whiteboard: p=1 → p=1 [qa-]
Comment 4•12 years ago
|
||
Please add STRs or mark it [qa-] if no QA verification is needed for this bug.
Comment 5•12 years ago
|
||
(In reply to Victor Carciu from comment #4)
> Please add STRs or mark it [qa-] if no QA verification is needed for this
> bug.
Sorry , I just observed that it is already marked as [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•