Commit 8e312461 authored by Mirek Klimos's avatar Mirek Klimos Committed by Facebook Github Bot

Revert DLOG_INFO warnings about RequestContext being overriden

Summary: This is causing a log spew on some services, let's remove it until we figure out where the problem is and fix

Differential Revision: D4171379

fbshipit-source-id: 2afdab6917a2498c376559f9171f6e93ab9c8ad7
parent 5b8add09
...@@ -115,8 +115,6 @@ class RequestContextScopeGuard { ...@@ -115,8 +115,6 @@ class RequestContextScopeGuard {
// Create a new RequestContext and reset to the original value when // Create a new RequestContext and reset to the original value when
// this goes out of scope. // this goes out of scope.
RequestContextScopeGuard() : prev_(RequestContext::saveContext()) { RequestContextScopeGuard() : prev_(RequestContext::saveContext()) {
DLOG_IF(INFO, prev_ != nullptr)
<< "Overriding folly::RequestContext - did you mean to unset it first?";
RequestContext::create(); RequestContext::create();
} }
...@@ -124,8 +122,6 @@ class RequestContextScopeGuard { ...@@ -124,8 +122,6 @@ class RequestContextScopeGuard {
// be automatically reset to the original value when this goes out of scope. // be automatically reset to the original value when this goes out of scope.
explicit RequestContextScopeGuard(std::shared_ptr<RequestContext> ctx) explicit RequestContextScopeGuard(std::shared_ptr<RequestContext> ctx)
: prev_(RequestContext::setContext(std::move(ctx))) { : prev_(RequestContext::setContext(std::move(ctx))) {
DLOG_IF(INFO, prev_ != nullptr && prev_.get() != RequestContext::get())
<< "Overriding folly::RequestContext - did you mean to unset it first?";
} }
~RequestContextScopeGuard() { ~RequestContextScopeGuard() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment