Reverted commit D3156698
authorMirek Klimos <miro@fb.com>
Wed, 20 Apr 2016 20:08:48 +0000 (13:08 -0700)
committerFacebook Github Bot 5 <facebook-github-bot-5-bot@fb.com>
Wed, 20 Apr 2016 20:20:40 +0000 (13:20 -0700)
commitbc9d9e8cfaceda506f7fe72dc0ce4b18d81dad2b
tree751efde3a149d6044d5ad8f13f31e6a5c7d97d79
parentaa088ebef6319194a0a762bc16629d5187ec2f00
Reverted commit D3156698

Summary:There're currently two ways to set RequestContext
- RequestContext::create() - creates new context and sets it
- RequestContext::setContext(context) - sets context previously captured by saveContext
In most cases, the RequestContext is set back after the request is processed but sometimes it's not (especially with RequestContext::create). We want to measure cpu time for a request by measuring the total cpu time when a RequestContext is set, so we need to make sure that it's properly reset after the thread is done with the request. Scope guards can help us with that.

Reviewed By: haijunz

Differential Revision: D3156698

fb-gh-sync-id: 7c3eb06c1cc27849071625011bf64c5ad36c0612
fbshipit-source-id: 7c3eb06c1cc27849071625011bf64c5ad36c0612
folly/io/async/Request.h