Skip to content

Commit ba2379a

Browse files
Jacob Steinfacebook-github-bot
authored andcommitted
Add setEnabledStatus to NoopAdditionalLogger
Summary: This diff adds getEnabledStatus and setEnabledStatus to the OSS NoopExternalConnectionLogger. Reviewed By: lenar-f Differential Revision: D39177020 fbshipit-source-id: 0d5e5468c7da657f344a34a6b4f51e6e0f20524d
1 parent 2187b59 commit ba2379a

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

third-party/mcrouter/src/mcrouter/lib/carbon/NoopAdditionalLogger.h

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,11 @@ class NoopExternalConnectionAdditionalLogger {
7979
bool shouldLog() const {
8080
return false;
8181
}
82+
83+
void setEnabledStatus(bool /* unused */) {}
84+
bool getEnabledStatus() {
85+
return false;
86+
}
8287
};
8388

8489
} // namespace carbon

third-party/mcrouter/src/mcrouter/mcrouter_config.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ class AdditionalExternalConnectionLogger
7272
: public carbon::NoopExternalConnectionAdditionalLogger {
7373
public:
7474
explicit AdditionalExternalConnectionLogger(
75-
carbon::ExternalCarbonConnectionLoggerOptions options)
75+
carbon::ExternalCarbonConnectionLoggerOptions& options)
7676
: NoopExternalConnectionAdditionalLogger(options) {}
7777
};
7878

0 commit comments

Comments
 (0)