Rob Hudson [:robhudson] Assignee
	Description • 11 years ago

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

	Rob Hudson [:robhudson] Assignee
	Comment 1 • 11 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.

	Rob Hudson [:robhudson] Assignee
	Comment 2 • 11 years ago

r? https://github.com/robhudson/zamboni/commit/d5e6343

	Rob Hudson [:robhudson] Assignee
	Comment 3 • 11 years ago

https://github.com/mozilla/zamboni/commit/aad6cc5

	Victor Carciu
	Comment 4 • 11 years ago

Please add STRs or mark it [qa-] if no QA verification is needed for this bug.

	Victor Carciu
	Comment 5 • 11 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-]